-
Notifications
You must be signed in to change notification settings - Fork 11
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
Contributions to the JTB part #55
Comments
@MarcMazas - thanks for helping. Contributions are welcome. As I see plugins is used in many places - https://github.com/mojohaus/javacc-maven-plugin/network/dependents So I think better will be improve it rather that introduce one new. Please mention me on PR, I will try to help with merge and next release. |
OK. |
For code formatting I will enable spotles in project and will be done automaticaly. As this is a Maven plugin some of information can be read here: if possible I would like to not use a static variable - it can introduce concurrency problems in multithread builds. |
spotless #58 |
Hi.
I'm the maintainer of JTB since JTB 1.4.0, which now is at 1.5.1. See https://github.com/jtb-javacc/JTB.
I plan to update the JTB distribution at Maven Central (https://central.sonatype.com/artifact/edu.purdue.cs/jtb).
I just saw that your javacc maven plugin has some features for handling JTB, but for the very old versions.
Would you like me to contribute to your maven plugin on the JTB part, or do you prefer I create and release a new dedicated plugin?
TIA.Marc
The text was updated successfully, but these errors were encountered: