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

feat: dependabot workflow automation for updating dependency #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Rajpratik71
Copy link

Signed-off-by: Pratik Raj [email protected]

@kaplanelad
Copy link
Contributor

Hey @Rajpratik71,
Can you please elaborate on what your motivation for adding is dependabot?

@Rajpratik71
Copy link
Author

Hey @Rajpratik71,
Can you please elaborate on what your motivation for adding is dependabot?

Hi @kaplanelad ,

Manual control of dependency is fine but with a growing no. of distributed upstream dependencies, it becomes hard to manage. So, for that automation should be there to update dependencies. Further, CI Pipeline is there to test those changes.

Further, this will not update the dependencies automatically, instead, a PR will be opened with changes that can be reviewed and tested with CI.

i.e why this automation will help here.

@kaplanelad
Copy link
Contributor

kaplanelad commented Nov 16, 2022

I'm trying to understand the pain of adding the dependencies updates. It means that we need 100% test coverage to ensure that nothing breaks and more things to cover.
Are you worried about vulnerable/malicious packages?

@Rajpratik71
Copy link
Author

I'm trying to understand the pain of adding the dependencies updates. It means that we need 100% test coverage to ensure that nothing breaks and more things to cover. Are you worried about vulnerable/malicious packages?

A PR will be opened with changes that can be reviewed and tested with CI.

If some dependencies updates break the CI then those dependencies can be skipped

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