Skip to content

fix(#851): remove redundant --update-snapshots option from mvnw params#867

Merged
yegor256 merged 2 commits intoobjectionary:masterfrom
volodya-lombrozo:851-remove-update-snapshots
Feb 25, 2026
Merged

fix(#851): remove redundant --update-snapshots option from mvnw params#867
yegor256 merged 2 commits intoobjectionary:masterfrom
volodya-lombrozo:851-remove-update-snapshots

Conversation

@volodya-lombrozo
Copy link
Copy Markdown
Member

This PR removes the redundant --update-snapshots option from mvnw.js.

Fixes #851

@volodya-lombrozo volodya-lombrozo force-pushed the 851-remove-update-snapshots branch from 1273028 to f468d24 Compare February 25, 2026 07:49
@volodya-lombrozo
Copy link
Copy Markdown
Member Author

@yegor256 could you have a look, please?

@yegor256 yegor256 merged commit beefe55 into objectionary:master Feb 25, 2026
16 checks passed
@0crat
Copy link
Copy Markdown

0crat commented Feb 25, 2026

@volodya-lombrozo Thanks for the contribution! You've earned +16 points following our policy - you received the base 16 points for reviews provided, plus points for your hits-of-code while staying within our quality guidelines. Your running score is +472; please keep them coming and don't forget to check your Zerocracy account too! 🚀

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.

Why do mvnw params specify --update-snapshots option?

3 participants