-
Notifications
You must be signed in to change notification settings - Fork 201
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
Introduce CompiledCGPatterns capturing the TS-Query and regex (placeholder) #527
Conversation
…older) [ghstack-poisoned]
…older) ghstack-source-id: cf2baf162fcbfc57e097e1fbc6e90c5ae2869b74 Pull Request resolved: #527
…gex (placeholder)" [ghstack-poisoned]
…older) ghstack-source-id: bf54c1bcedd104b1403acef426fc1a11974cd82d Pull Request resolved: #527
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.
Minor documentation nit (also, might be worth having a full documentation comment for both get_match
and get_matches
)
src/models/capture_group_patterns.rs
Outdated
} | ||
|
||
impl CompiledCGPattern { | ||
/// Applies the query upon the given node, and gets all the matches |
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.
Is the documentation correct here? It seems this returns 0 or 1 matches. Is this meant to be the docstring for get_matches
?
…gex (placeholder)" Introduces `CompiledCGPattern` which wraps `treesitter::Query` and `regex::Regex` (A simple temporary test to show that regex support is added but not supported. This test is updated in the stacked diff). ------ [ghstack-poisoned]
…gex (placeholder)" Introduces `CompiledCGPattern` which wraps `treesitter::Query` and `regex::Regex` (A simple temporary test to show that regex support is added but not supported. This test is updated in the stacked diff). ------ [ghstack-poisoned]
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.
👍
…older) ghstack-source-id: 0ef1c1ae7eaf0b3c2d49a0287818b1b2677b33a0 Pull Request resolved: #527
…older) ghstack-source-id: 0ef1c1ae7eaf0b3c2d49a0287818b1b2677b33a0 Pull Request resolved: uber/piranha#527
Introduces
CompiledCGPattern
which wrapstreesitter::Query
andregex::Regex
(A simple temporary test to show that regex support is added but not supported. This test is updated in the stacked diff).
Stack from ghstack (oldest at bottom):