chore: readd dependabot, including a way to refresh the project files for all the modules#2997
Merged
mdelapenya merged 26 commits intotestcontainers:mainfrom Mar 11, 2025
Merged
Conversation
✅ Deploy Preview for testcontainers-go ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
mdelapenya
commented
Feb 26, 2025
Member
Author
|
@mmorel-35 as contributor of the modulegen, would you like to take a look at this? 🙏 |
…ntry per Go module
stevenh
requested changes
Feb 27, 2025
stevenh
reviewed
Feb 27, 2025
Contributor
stevenh
left a comment
stevenh
left a comment
There was a problem hiding this comment.
Some questions before I continue on review.
* main: docs: refine texts on how to set the module image (testcontainers#3012) feat(modules): add dind module (testcontainers#3004) docs: correct container variable (testcontainers#3010)
mdelapenya
added a commit
to mdelapenya/testcontainers-go
that referenced
this pull request
Mar 12, 2025
* main: chore(deps): bump github.com/opencontainers/image-spec from 1.1.0 to 1.1.1, dario.cat/mergo from 1.0.0 to 1.0.1 (testcontainers#3030) chore(deps): bump github/codeql-action from 3.28.0 to 3.28.11 (testcontainers#3014) chore(deps): bump ossf/scorecard-action from 2.4.0 to 2.4.1 (testcontainers#3013) chore: readd dependabot, including a way to refresh the project files for all the modules (testcontainers#2997)
mdelapenya
added a commit
to mdelapenya/testcontainers-go
that referenced
this pull request
Mar 13, 2025
* main: chore(deps): bump github.com/opencontainers/image-spec from 1.1.0 to 1.1.1, dario.cat/mergo from 1.0.0 to 1.0.1 (testcontainers#3030) chore(deps): bump github/codeql-action from 3.28.0 to 3.28.11 (testcontainers#3014) chore(deps): bump ossf/scorecard-action from 2.4.0 to 2.4.1 (testcontainers#3013) chore: readd dependabot, including a way to refresh the project files for all the modules (testcontainers#2997) docs: refine texts on how to set the module image (testcontainers#3012) feat(modules): add dind module (testcontainers#3004) docs: correct container variable (testcontainers#3010) chore: update Weaviate version to v1.29.0 and Weaviate Go client to v5.0.2 (testcontainers#3006) chore(deps): bump docker/setup-docker-action from 4.1.0 to 4.2.0 (testcontainers#3002) docs: fix typo in Grafana lgtm module (testcontainers#2999) feat: add ScyllaDB module (testcontainers#2992) feat(mssql): add WithInitSQL function (testcontainers#2988)
mdelapenya
added a commit
to mdelapenya/testcontainers-go
that referenced
this pull request
Mar 13, 2025
* main: chore(deps): bump github.com/opencontainers/image-spec from 1.1.0 to 1.1.1, dario.cat/mergo from 1.0.0 to 1.0.1 (testcontainers#3030) chore(deps): bump github/codeql-action from 3.28.0 to 3.28.11 (testcontainers#3014) chore(deps): bump ossf/scorecard-action from 2.4.0 to 2.4.1 (testcontainers#3013) chore: readd dependabot, including a way to refresh the project files for all the modules (testcontainers#2997) docs: refine texts on how to set the module image (testcontainers#3012) feat(modules): add dind module (testcontainers#3004) docs: correct container variable (testcontainers#3010)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
What does this PR do?
This PR enables dependabot for all the modules in the project so that they receive updates when needed.
It also adds the automation for creating dependabot updates for the new modules added with the code generation tool, and a new
refreshcommand to refresh all the project files:As a consequence, the unit tests for the modulegen module has been updated (simplified?) making them not affecting the current workspace: for that, the project and module files will be copied to a temp dir, and the test context will happen there, not adding dirty to the current workspace.
Finally, the dependabot file has been excluded from the automation to detect changed, so that a change on it does not include all the modules in the CI pipeline.
Why is it important?
Given the CI pipeline is now more efficient, only running the modified modules, now it's possible to handle dependabot updates not consuming all the GHA runners at a time. Instead just 4 runners will be used for testing the changes: detect + lint + testx2 + end, which is way faster than before, where the entire pipeline had to be built (~100 workers), adding flakiness.