-
Notifications
You must be signed in to change notification settings - Fork 0
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
feat!: moved bunx and dockertestx out of the root modules #79
Conversation
32e78be
to
4a3e769
Compare
4a3e769
to
7c6faf0
Compare
@@ -8,9 +8,9 @@ jobs: | |||
release-please: | |||
runs-on: ubuntu-latest | |||
steps: | |||
- uses: GoogleCloudPlatform/release-please-action@v2 | |||
- uses: googleapis/release-please-action@v4 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The GoogleCloudPlatform/release-please-action@v2
is outdated. We now need to create release tags with directory prefixes (e.g., dockertestx/v0.1.7
). This has been tested in PR #85.
When a new version like v1.0.0 is suggested, we can manually specify the desired next version.
For more details, see: https://github.com/googleapis/release-please?tab=readme-ov-file#how-can-i-fix-release-notes
27a0582
to
2566413
Compare
fb3ffe0
to
0f0121f
Compare
44287f2
to
29a2b6f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Moved bunx and dockertestx out of the root modules since they have many dependencies that most users don't need.
Users can install these modules using the following commands: