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 #153

Merged
merged 3 commits into from
Nov 27, 2022
Merged

Add dependabot #153

merged 3 commits into from
Nov 27, 2022

Conversation

staticdev
Copy link
Collaborator

@staticdev staticdev commented Nov 22, 2022

  • Dependabot will automatically check dependencies and propose PRs.

@staticdev staticdev added the ci Continuous Integration label Nov 22, 2022
@staticdev staticdev requested a review from aalaesar November 22, 2022 06:17
@staticdev
Copy link
Collaborator Author

Lgtm @aalaesar ?

@aalaesar
Copy link
Member

aalaesar commented Nov 24, 2022

@staticdev isn't that tool overkill for a simple ansible role? What is the use case ?

@staticdev
Copy link
Collaborator Author

staticdev commented Nov 24, 2022

@aalaesar do not think it is overkill. I have it in ALL my Github projects (and many of them are ansible roles). In this repo, it would find updates in Github Actions which are very frequent and also in versions of libraries we use. I think it is very important to keep the role fresh.

One good example, this week ansible 7 was released and I discovered many of my roles need adjustments to work with it because dependabot created updates for it.

@aalaesar
Copy link
Member

Ok I understand the usefullness of the tool, thank you for putting light on it 😃
However, seeing this, I doubt it is usefull in this case, unless there is a dependency manifest I forgot somewhere.
Or maybe I just can't project the example you gave me because I don't know the tool. 😕 Is it for molecule ?

@staticdev
Copy link
Collaborator Author

@aalaesar you are right that it could be better if it had direct support to Galaxy, though it is still useful for dependencies used to run molecule and also to update the existing actions in the workflows. As you see for example in the last commit Labeler action is failing on version 4.0.0, current version is 4.1.0.. it is possible that we will not have problems with the new version. Last but not least it also updates ansible-lint and yamllint that has been updated frequently lately.

Copy link
Member

@aalaesar aalaesar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok then.

@staticdev staticdev merged commit eb4f698 into nextcloud:main Nov 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci Continuous Integration
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants