Create weekly test releases#310
Merged
copybara-staging[bot] merged 1 commit intomasterfrom Feb 7, 2025
Merged
Conversation
2d6a5c9 to
dea02fc
Compare
alexeagle
reviewed
Jan 23, 2025
alexeagle
approved these changes
Jan 23, 2025
Contributor
alexeagle
left a comment
There was a problem hiding this comment.
Thank you for working on this!
.github/workflows/release.yaml
Outdated
|
|
||
| - name: Get current date | ||
| id: date | ||
| run: echo "::set-output name=date::$(date +%Y%m%d)" |
Contributor
There was a problem hiding this comment.
this is deprecated https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
they want you to use GITHUB_OUTPUT instead
.github/workflows/release.yaml
Outdated
| runs-on: ubuntu-latest | ||
| steps: | ||
| - name: Checkout code | ||
| uses: actions/checkout@v3 |
Contributor
There was a problem hiding this comment.
nit: there is a v4 - since this is a new workflow you probably want the latest of the deps
dea02fc to
14f6c28
Compare
c9bf99b to
6df68cd
Compare
This change adds automated weekly test releases that don't require to compile the code. An additional option is added to do manual releases. Fixes #275 Change-Id: If038652623f8a8e0528984c9ed53a97d2cfedd75
489737d to
d81d656
Compare
alexeagle
reviewed
Feb 8, 2025
| java-version: '21' | ||
|
|
||
| - name: Build with Bazel | ||
| run: bazel build java/com/google/copybara/copybara_deploy.jar |
Contributor
There was a problem hiding this comment.
I think this needs --stamp or --embed_label here, since today's new release prints
Feb 07, 2025 3:52:34 PM com.google.copybara.Main configureLog
INFO: Setting up LogManager
Copybara source mover (Version: Unknown version)
Collaborator
Author
There was a problem hiding this comment.
I have to look at this, our current oss implementation does this:
protected String getVersion() {
return "Unknown version";
}
I have to match to what we do internally.
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
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.
Fixes #275