-
Notifications
You must be signed in to change notification settings - Fork 142
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
BinarizeParseMojo generalization #2672
BinarizeParseMojo generalization #2672
Conversation
@levBagryansky @maxonfjvipon could you check this one, please? |
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.
@c71n93 please check
* think we can implement it, using something like {@code FFINodeFactory}, that will return | ||
* appropriate FFI node for every XML node from {@code nodes}. Also it will be great to move | ||
* paths to XML FFI insert nodes (such as {@code "/program/rusts/rust"}) from this method to | ||
* a class field. |
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.
@c71n93 we can specify "static field" here
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.
@c71n93 LGTM! Thanks
@yegor256 could you check this one, please? |
1 similar comment
@yegor256 could you check this one, please? |
@rultor merge |
Closes #2649
PR-Codex overview
Detailed summary
BinarizeParseMojo
class in theeo-maven-plugin
project.addRust
method toaddFFIs
.