-
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
ES-2153 - Added support for multiple files and for updating default parameters #426
Conversation
@Gaardsholt do you think there will be timeout issues because of how long it can take to run this now? |
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.
Can you please write a proper title for the pull request, just to make the release notes a bit better
I'm not completely done reviewing this pull request, I will continue when you have addressed the comments I have added so far :)
…- BREAKING change Added defaults for directory and schedule
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.
LGTM - kinda hard to follow the changes, but I trust you :)
Could we add test cases for this new functionality?
Co-authored-by: Lasse Gaardsholt <lasse.gaardsholt@bestseller.com>
|
Added support for multiple files.
directory
is set to a folder, dependabot-circleci will travel the whole directory tree and check all yml/yaml files.directory
is now.circleci/config.yml
. This different than the old behavior and will break stuff in our templates.Added support for updating the default value of parameters
Example:
https://github.com/BESTSELLER/cci-common-test/pulls
Files that can't be updated:
https://github.com/BESTSELLER/cci-common-test/blob/49ca40c5737faf0750552dfeeb7b53c58ae8e51d/src/jobs/build_n_push_docker.yml#L11
This will fix: #194