-
Notifications
You must be signed in to change notification settings - Fork 267
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
Create weekly test releases #310
Conversation
2d6a5c9
to
dea02fc
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.
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)" |
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.
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 |
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.
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
java-version: '21' | ||
|
||
- name: Build with Bazel | ||
run: bazel build java/com/google/copybara/copybara_deploy.jar |
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.
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)
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.
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.
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.
Just merged eee4901 for this
Fixes #275