This repository has been archived by the owner on Dec 15, 2022. It is now read-only.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🚨 WIP - Do Not Merge 🚨
Description of the Change
This PR seeks to implement the proposal described at https://github.com/atom/symbols-view/wiki/v1-Proposal.
Benefits
symbols-view
, by providing grammar-specific strategies for navigation to symbols definitions or referencessymbols-view
will be encapsulated in a provider, allowing for further enhancement ofsymbols-view
in ways that might go beyond what ctags can allowPossible Drawbacks
Many packages may be directly integrating with
symbols-view
by activating the package and attempting to use functions that are not part of a public API. Such packages will need to be updated to continue to function correctly.Applicable Issues
Tasks
Goto Definition Request
Find References Request
Document Symbols Request
CtagsProvider
and register it like any other providergo-to-view.js
to useetch
go-back-view.js
to useetch
/cc @zmb3
🚨 WIP - Do Not Merge 🚨