-
Notifications
You must be signed in to change notification settings - Fork 188
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
Would this be possible in a framework like Ruby or Rails #80
Comments
It is possible to implement a variant of Piranha for ruby using Parser. As for your question on finding references to a particular flag, it depends on the coding style, the availability of tools to perform alias analysis, etc. If you can share some examples on expected refactorings, I can clarify further. |
@Bennet-Sunder This can now be done by adding support for Ruby in polyglot-piranha . |
@mkr-plse @ketkarameya just as a heads up I have submitted a draft PR for adding support for ruby. Will clean it up and mark it ready for review by next week. |
Hello, @Bennet-Sunder what's the progress on adding support for ruby? |
Let us know since our org would like to integrate with Piranha, pending with that integration |
@juamanti it should be merged shortly, you can follow the PR for updates on when it'll be merged. In the meanwhile you can evaluate it using my fork. |
Will do! |
Thank you so much!! cc: @Charan Mahesan ***@***.***>
Best, Juan
…On Mon, May 20, 2024 at 2:18 PM Bennet Sunder ***@***.***> wrote:
@juamanti <https://github.com/juamanti> support for ruby is now merged (
#649 <#649>) and it should be
available in the next release.
—
Reply to this email directly, view it on GitHub
<#80 (comment)>, or
unsubscribe
<https://github.com/notifications/unsubscribe-auth/AXLS7NNAM5OSRAEJSUA6OKTZDI455AVCNFSM4PF5VXX2U5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TEMJSGA4TMMZXGI3Q>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Considering that ruby is not statically typed and has meta-programming where it's hard to find the references of a particular flag. Just curious about the possibilities.
The text was updated successfully, but these errors were encountered: