-
-
Notifications
You must be signed in to change notification settings - Fork 2k
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
Enable reproducible builds #2641
Conversation
Hey thanks for wanting to help out! Did you also test the reproducibility? We have some custom code generation and because support for modules in Maven is still very janky It would also be nice if we could add this to CI to ensure we don't break it in the future. |
Codecov Report
@@ Coverage Diff @@
## main #2641 +/- ##
=========================================
Coverage 84.77% 84.77%
Complexity 2686 2686
=========================================
Files 322 322
Lines 9530 9530
Branches 903 903
=========================================
Hits 8079 8079
Misses 1120 1120
Partials 331 331 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
60eb577
to
e2cc35c
Compare
I tested the reproducibility by running:
which is the first level of tests = 2 local builds
Reproducible Builds will be tested on future releases by https://github.com/jvm-repo-rebuild/reproducible-central |
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.
Looks good.
I've included the plugin version changes in cucumber-parent
instead. This should avoid the need to manually set versions in this project.
The build will eventually pass once cucumber-parent
arrives on Maven Central.
Do note that Cucumber-JVM depends on several components that are build elsewhere in the Cucumber org. These include:
- https://github.com/cucumber/tag-expressions
- https://github.com/cucumber/messages/
- https://github.com/cucumber/cucumber-expressions/
- https://github.com/cucumber/html-formatter
- https://github.com/cucumber/ci-environment
- https://github.com/cucumber/gherkin
Will you also make these reproducible?
Hi @hboutemy, Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾 In return for this generous offer we hope you will:
On behalf of the Cucumber core team, |
🤔 What's changed?
see https://maven.apache.org/guides/mini/guide-reproducible-builds.html
⚡️ What's your motivation?
Reproducible Builds
🏷️ What kind of change is this?
♻️ Anything particular you want feedback on?
📋 Checklist:
This text was originally generated from a template, then edited by hand. You can modify the template here.