-
Notifications
You must be signed in to change notification settings - Fork 343
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
ci: Use the release-metadata action #597
Conversation
janbuchar
commented
Oct 16, 2024
•
edited
Loading
edited
- closes Deprecated variables in git_cliff_core::changelog #598
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.
If issue is not linked to the pull request then estimate the pull request!
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.
If issue is not linked to the pull request then estimate the pull request!
lint_check: | ||
name: Lint check | ||
uses: apify/workflows/.github/workflows/python_lint_check.yaml@main | ||
|
||
type_check: | ||
name: Type check | ||
uses: apify/workflows/.github/workflows/python_type_check.yaml@main | ||
|
||
unit_tests: | ||
name: Unit tests | ||
uses: apify/workflows/.github/workflows/python_unit_tests.yaml@main |
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.
On a manual release, this really seems redundant. Also, see #469
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.
Currently, it is not redundant, since we don't have any additional trigger of code checks for the master (only for PRs).
I'm thinking if |
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.
few questions from my end.
also the changelog changes look wrong, there shouldn't be any, right?
.github/workflows/pre_release.yaml
Outdated
# Trigger a beta version release (pre-release) on push to the master branch. | ||
push: | ||
branches: | ||
- use-release-metadata-action # TODO |
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.
how is this supposed to be working? canary builds should be done only on the default branch
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.
that's so that I can test the workflow against this branch, I intend to change it to master
before merging
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 just don't like the name "release-metadata" 😄, otherwise LGTM.
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