Skip to content
This repository has been archived by the owner on Feb 26, 2024. It is now read-only.

delete dependabot.yml #5604

Closed
wants to merge 1 commit into from
Closed

delete dependabot.yml #5604

wants to merge 1 commit into from

Conversation

davidmurdoch
Copy link
Member

The existence of this file with this configuration disables all security updates. See: dependabot/dependabot-core#5845

The existence of this file with this configuration disables all security updates. See: dependabot/dependabot-core#5845
Comment on lines -18 to -20
# Disable version updates for npm dependencies
# see https://docs.github.com/en/code-security/dependabot/dependabot-security-updates/configuring-dependabot-security-updates#overriding-the-default-behavior-with-a-configuration-file
open-pull-requests-limit: 0
Copy link
Member

Choose a reason for hiding this comment

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

@davidmurdoch what happens if this setting is removed?

Copy link
Member Author

Choose a reason for hiding this comment

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

The problem with the current config is that github doesn't allow us to specify npm twice. You are looking at the 2nd of 2 npm configs. This open-pull-requests-limit: 0 setting is the official documented hack way to enable security-only updates; setting open-pull-requests-limit actually sets it to 10 if the Security Updates setting is enabled in the GitHub UI.

@haltman-at
Copy link
Contributor

This is no longer necessary, right?

@davidmurdoch
Copy link
Member Author

This is no longer necessary, right?

Have we confirmed that hack from the other PR has re-enabled dependabot?

@haltman-at
Copy link
Contributor

haltman-at commented Oct 12, 2022

Yup! I've been running it to find dependencies to update and even made one update on that basis (#5613). :)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants