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

Guard against writing empty dependabot.yml files #177

Merged
merged 1 commit into from
Feb 9, 2021

Conversation

ekcasey
Copy link
Member

@ekcasey ekcasey commented Feb 9, 2021

Summary

Dependabot requires updates to be an array and it will fail if omitted or empty. The dependabot.yml file will need to be manually deleted from repos where it is current empty or no longer needed (now that we no longer subscribe to GHA updates) because the pipeline builder does not delete files.

Use Cases

This affect repos that previously had GHA in their dependabot update config and now do not require dependabot at all (build package and builder repos).

Checklist

  • I have viewed, signed, and submitted the Contributor License Agreement.
  • I have added an integration test, if necessary.

Dependabot requires updates to be an array and it will fail if omitted or empty. The dependabot.yml file will need to be manually deleted from repos where it is current empty or no longer needed (now that we no longer subscribe to GHA updates) because the pipeline builder does not delete files.

Signed-off-by: Emily Casey <ecasey@vmware.com>
@ekcasey ekcasey added type:bug A general bug semver:patch A change requiring a patch version bump labels Feb 9, 2021
@ekcasey ekcasey requested a review from a team February 9, 2021 23:21
@ekcasey ekcasey merged commit 72a7b70 into main Feb 9, 2021
@ekcasey ekcasey deleted the bugfix/empty-dependabot-config branch February 9, 2021 23:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
semver:patch A change requiring a patch version bump type:bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant