-
Notifications
You must be signed in to change notification settings - Fork 1
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
Make it easier to update gradle verification reference #68
Conversation
e991fb9
to
3b9e268
Compare
bitrise.yml
Outdated
test: | ||
after_run: |
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.
Since check_gradle_verification
contains only bundle::generate_gradle_verification_reference: { }
, and that is also explicitly added here, it will be run twice. Why do we need this after_run?
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.
The idea was to check if it is up-to-date even when running the test WF locally.
bitrise.yml
Outdated
- bundle::generate_gradle_verification_reference: { } | ||
|
||
step_bundles: | ||
generate_gradle_verification_reference: |
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.
Since this also performs a check I'd call this check_gradle_verification_reference
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.
renamed.
check_gradle_verification
) that can be used to update verification-metadata.xml.This is now included when running tests, to make sure it is kept up-to-date.