-
Notifications
You must be signed in to change notification settings - Fork 612
Implement Spanned
to retrieve source locations on AST nodes
#1435
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
Merged
Merged
Changes from 15 commits
Commits
Show all changes
38 commits
Select commit
Hold shift + click to select a range
1a77bac
feat(tokenizer): add source location spans to tokens
Nyrox d818012
feat: begin work on trait Spanned
Nyrox 079a4e2
implement a bunch more stuff
Nyrox b97a781
Merge branch 'feat/ast-source-locations'
Nyrox df9ab1e
fix: restore old behaviour of location display
Nyrox b718c76
implement spans for eveeeeen more ast nodes
Nyrox 8986a1e
feat: more ast nodes
Nyrox aeb4f3a
start working on better tests
Nyrox 4de3209
feat: implement spans for Wildcard projections
Nyrox a04888a
make union_spans public
Nyrox 1b2b03d
enable serde feat for spans and locations
Nyrox 5f60bdc
feat: implement remaining ast nodes
Nyrox ea8a6b1
fix unused variable warnings
Nyrox 6a9250a
undo parse_keyword signature change
Nyrox 0804e99
fix: diverging hash and partialeq implementations
Nyrox eb9ff9a
Update src/ast/spans.rs
Nyrox a93cebc
improve docs & un-pub union_spans
Nyrox 734264a
move union_spans to top of file
Nyrox 441ceb1
replace old tests
Nyrox e6a4340
pr feedback
Nyrox 16a3f2a
add small comment
Nyrox 98b051d
refactor: rewrite all span implementations to pattern match exhaustiv…
Nyrox d76d1e0
for_clause is mssql, not mysql
Nyrox bf75fe4
Merge branch 'main' into main
Nyrox 71c27ea
cargo fmt
Nyrox 1353bf2
Merge remote-tracking branch 'apache/main'
Nyrox a31c6a6
lint & no-std
Nyrox ce8b35c
add IgnoreField helper
Nyrox 2bb72a4
docs and fixes
Nyrox 4dda1fe
fix: test failing
Nyrox 3fa3766
pr feedback
Nyrox 6bfe13f
rename ignore_field.rs -> attached_token.rs
Nyrox 903f24a
pr feedback
Nyrox 3989efe
add AttachedToken::empty
Nyrox b2b8795
Merge branch 'main' of https://github.com/apache/datafusion-sqlparser-rs
Nyrox 1f3d514
Merge remote-tracking branch 'apache/main' into Nyrox/main
alamb 23c4922
update test to avoid overflow
alamb b24c9fe
Merge branch 'main' of https://github.com/apache/datafusion-sqlparser-rs
Nyrox 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 hidden or 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
This file contains hidden or 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.
Uh oh!
There was an error while loading. Please reload this page.