Skip to content
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

Merged
merged 4 commits into from
Jul 10, 2023

Conversation

ketkarameya
Copy link
Collaborator

@ketkarameya ketkarameya commented Jul 5, 2023

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).


Stack from ghstack (oldest at bottom):

ketkarameya added a commit that referenced this pull request Jul 5, 2023
…older)

ghstack-source-id: cf2baf162fcbfc57e097e1fbc6e90c5ae2869b74
Pull Request resolved: #527
ketkarameya added a commit that referenced this pull request Jul 5, 2023
…older)

ghstack-source-id: bf54c1bcedd104b1403acef426fc1a11974cd82d
Pull Request resolved: #527
@ketkarameya ketkarameya mentioned this pull request Jul 5, 2023
Copy link
Contributor

@lazaroclapp lazaroclapp left a 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)

}

impl CompiledCGPattern {
/// Applies the query upon the given node, and gets all the matches
Copy link
Contributor

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]
@ketkarameya ketkarameya mentioned this pull request Jul 10, 2023
Copy link
Contributor

@lazaroclapp lazaroclapp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@ketkarameya ketkarameya merged commit 45bca71 into gh/ketkarameya/25/base Jul 10, 2023
5 checks passed
ketkarameya added a commit that referenced this pull request Jul 10, 2023
…older)

ghstack-source-id: 0ef1c1ae7eaf0b3c2d49a0287818b1b2677b33a0
Pull Request resolved: #527
@ketkarameya ketkarameya deleted the gh/ketkarameya/25/head branch July 10, 2023 18:54
crystalninja5 added a commit to crystalninja5/solid-tribble that referenced this pull request Aug 11, 2024
…older)

ghstack-source-id: 0ef1c1ae7eaf0b3c2d49a0287818b1b2677b33a0
Pull Request resolved: uber/piranha#527
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants