-
Notifications
You must be signed in to change notification settings - Fork 2
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Basic Goto Declaration Support #22
Closed
Closed
Changes from 1 commit
Commits
Show all changes
24 commits
Select commit
Hold shift + click to select a range
e23da49
Analyze entire project on opening it
kerristrasz e0cc38b
handle workspace events
kerristrasz 07bd76d
Fix text document modification detection
kerristrasz 380e65c
Add caching for tree sitter analysis
kerristrasz 683af77
Add prettier config
kerristrasz e6f5b3b
Steal representation of Modelica projects from OMFrontend
kerristrasz 38da1aa
Finish resolve algorithm; integrate with analyzer and server
kerristrasz 3646148
Get outline working again; changes to uri handling
kerristrasz 7eb3727
Fix infinite recursion and declaration symbols not being found
PaddiM8 9dae5cb
Implement goto declaration/definition for locals
kerristrasz 8795ed1
Minor tweaks to resolve algorithm
kerristrasz 30cbf1d
Add new resolve algorithm
kerristrasz df86c40
Fix resolution of types and locals
PaddiM8 c15d93f
Fix resolution of inherited members
PaddiM8 aaa6dbc
Allow relative references in extends clauses
PaddiM8 6101ada
Bugfixes; use paths internally
kerristrasz e015766
Fix endless recursion bug in resolveReference
kerristrasz 2d0cc19
Fix resolution algorithm
kerristrasz b597f9a
Tweak resolution algorithm
kerristrasz de6798c
Reanalyse when a file is modified, before it's saved
PaddiM8 634fb3c
Incremental parsing
PaddiM8 cf6d053
Whitespace changes
AnHeuermann 6a727b3
Add additional documentation.
kerristrasz 38dc70e
update settings
kerristrasz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think @AnHeuermann meant that you can add this to your VS code settings so the code you write is always formatted without trailing whitespaces. I don't think he meant to add these settings to the project.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed 😆
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Surely it would be a good idea to add these to the project settings anyways, right? Doing that would help to ensure a more consistent style across contributors.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not everyone uses VS Code, so I usually don't add the .vscode directory. For a VS Code extension this might be different.
Both is fine with me.