-
Notifications
You must be signed in to change notification settings - Fork 889
chore: Remove dependabot #1667
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
Comments
What's the alternative to this functionality? It serves a useful purpose. Is there a way to achieve the same goal less noisily? Perhaps we run it less often or have it bundle all changes for a period into a single PR (not sure it can), or something else? |
Why do we need this functionality? |
It keeps us from using dependencies that have vulnerabilities patched in later versions or that have passed their end of life maintenance status. Probably gets us bug fixes as well. |
There's a trade off between bug fixes and PR noise. You may be surprised by how few bugs dependabot solves. But, the trade off is difficult to quantify. Since it primarily updates FE dependencies, I think the resolution is best left to someone on our FE team. |
I haven't seen evidence that this causes problems. Maybe we can roll all the changes into a different branch: I'd want that branch to have a datestamp in its name, and I don't see a way to request merging that branch. I'm sure someone out there has scripts to do stuff like this. |
I removed labels from dependabot here: #2110. This should help make it less visually spammy. I agree with Ketan that dependabot is useful for vulnerabilities. This is a won't fix for now! |
cc @vapurrmaid
The majority of our pull requests are from dependabot making it annoying to sift through Pull Requests. Also, unless dependabot is submitting a security fix, an upgrade may just break the code (see Minimum Version Selection).
The text was updated successfully, but these errors were encountered: