Skip to content
This repository has been archived by the owner on Dec 13, 2018. It is now read-only.

Commit

Permalink
Enable outline view
Browse files Browse the repository at this point in the history
Summary:
Requires building cquery on branch pelmers/containername-change to look correct.

The LSP specification specifically warns against using documentSymbols containerName fields to create symbol hierarchies but that's exactly what we do in the outline view. In the future I think there would be better ways to do this.
https://github.com/Microsoft/language-server-protocol/blob/master/protocol.md#textDocument_documentSymbol
and discussion at microsoft/language-server-protocol#136

Reviewed By: a20012251

Differential Revision: D6451986

fbshipit-source-id: 9182535584910f82039662951a1d2b9e461b47d8
  • Loading branch information
pelmers authored and facebook-github-bot committed Dec 1, 2017
1 parent f8c05db commit 6a3c68b
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions pkg/nuclide-cquery-lsp/lib/main.js
Original file line number Diff line number Diff line change
Expand Up @@ -353,6 +353,12 @@ class Activation {
analyticsEventName: 'cquery.getActions',
applyAnalyticsEventName: 'cquery.applyAction',
},
outline: {
version: '0.1.0',
analyticsEventName: 'cquery.outline',
updateOnEdit: true,
priority: 1,
},
};

const languageService = new AtomLanguageService(
Expand Down

0 comments on commit 6a3c68b

Please sign in to comment.