-
Notifications
You must be signed in to change notification settings - Fork 15
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
build(deps): update dependency org.codehaus.mojo:flatten-maven-plugin to v1.3.0 #507
build(deps): update dependency org.codehaus.mojo:flatten-maven-plugin to v1.3.0 #507
Conversation
c5ade45
to
49c7f7a
Compare
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
49c7f7a
to
0d5ab0e
Compare
Merge-on-green attempted to merge your PR for 6 hours, but it was not mergeable because either one of your required status checks failed, one of your required reviews was not approved, or there is a do not merge label. Learn more about your required status checks here: https://help.github.com/en/github/administering-a-repository/enabling-required-status-checks. You can remove and reapply the label to re-run the bot. |
@meltsufin I wrote the analysis that the validation result is false positive #507 (comment) . The flatten plugin new version is working correctly. The validation logic is wrong. |
Can you create an issue for this? Would you be able to provide a fix so that we can upgrade the plugin? |
Issue #525 The fix in the dependencies.sh (templated in OwlBot post processor): https://github.com/googleapis/synthtool/pull/1663/files |
This should become green after googleapis/java-storage#1732 is merged to the main branch and we retry the "downstream / dependencies (11, java-storage)" check. |
I'm rebasing this pull request to see flatten plugin check. |
8c78152
to
63cc831
Compare
Good. It failed.
|
This PR contains the following updates:
1.2.7
->1.3.0
Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
This PR has been generated by Mend Renovate. View repository job log here.
BEGIN_COMMIT_OVERRIDE
deps: update dependency org.codehaus.mojo:flatten-maven-plugin to v1.3.0
This resolves the problem of test-scope dependencies unexpectedly excluded.
END_COMMIT_OVERRIDE