Skip to content
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

Add dependabot config #61

Merged
merged 2 commits into from
Jul 8, 2023
Merged

Conversation

felixfontein
Copy link
Contributor

No description provided.

@webknjaz
Copy link
Member

webknjaz commented Jul 6, 2023

@felixfontein why do you think this is needed? I'd rather keep major-tag-driven versioning in the smoke tests to keep the noise down in such a low-traffic project. Besides, the only action it'd bump is the official actions/checkout@v3. Is it worth it?

@felixfontein
Copy link
Contributor Author

@webknjaz I usually add this config to all my projects so I won't forget about updating things like actions/checkout@v3 to a newer version (and I don't have to do it manually). Also these updates only happen very rarely, so this is only very seldomly resulting in something to do.

You have to decide whether you find this useful; I personally do, but if you rather want to notice the version change yourself, that's up to you.

(It would be nicer if dependabot would also check the action itself, but I haven't yet figured out how to let it do that...)

@webknjaz
Copy link
Member

webknjaz commented Jul 7, 2023

I don't like when it bumps the patch versions but if it could do just v3->v4, that'd be acceptable.

@felixfontein
Copy link
Contributor Author

I've never seen it do that. I guess it only does it if you start having patch versions in the actions in the first place (and IMO then it makes sense to update these).

@webknjaz
Copy link
Member

webknjaz commented Jul 8, 2023

Okay, let's see how it goes..

@webknjaz webknjaz merged commit 6070eaf into ansible-community:main Jul 8, 2023
@felixfontein felixfontein deleted the dependabot branch July 8, 2023 21:05
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.

2 participants