Skip to content

Add dependabot version scanning config - #132

Draft
brikin01 wants to merge 1 commit into
secure-buildfrom
add-dependabot-version-scanning
Draft

Add dependabot version scanning config#132
brikin01 wants to merge 1 commit into
secure-buildfrom
add-dependabot-version-scanning

Conversation

@brikin01

@brikin01 brikin01 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@NeethuESim NeethuESim left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

@brikin01 one danger I see with this version package update is regression issues. Existing flows might break because of incompatibility between packages after upgrade. Can you add some dependency here that update should be made only after the integration & unit tests pass with these changes ? Or if these changes are made after PR creation, then merge should be allowed only if all the integration & unit tests pass.

@brikin01

brikin01 commented Aug 6, 2026

Copy link
Copy Markdown
Collaborator Author

@brikin01 one danger I see with this version package update is regression issues. Existing flows might break because of incompatibility between packages after upgrade. Can you add some dependency here that update should be made only after the integration & unit tests pass with these changes ? Or if these changes are made after PR creation, then merge should be allowed only if all the integration & unit tests pass.

The typical flow is:

  1. Dependabot runs on a weekly cadence
  2. If it detects a version can/should be updated, it will open a PR for those updates, thus triggering any CI workflows we have.
  3. We still then need to manually review and merge it.

So I think there shouldn't be a problem there, in that our tests should catch any regressions before we ever merge. And from there we could either close the Dependabot PR or figure out how to use the latest version without breaking CI. Any rules around requiring tests to pass before merging will just be a matter of having stricter rules on our main branch, which I believe is part of the work of STESOL-557.

@NeethuESim

Copy link
Copy Markdown
Collaborator

STESOL-557

cool. STESOL-557 will prevent any accidental merging for any PRs without the integration/unit tests passing.

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