Skip to content
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: add workflow for uploading installers to release assets #208

Merged
merged 10 commits into from
Feb 7, 2023

Conversation

AnqiPang
Copy link
Contributor

@AnqiPang AnqiPang commented Feb 4, 2023

Issue #, if available:

Description of changes:
This workflow is to be run after the release-installer.yaml workflow completes. And after this workflow completes, the next step is to run the release-homebrew.yaml workflow. All the three workflows should be merged together once this issue is fixed.
Testing done:
Tested the workflow within my own fork.

  • I've reviewed the guidance in CONTRIBUTING.md

License Acceptance

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

BEGIN_COMMIT_OVERRIDE
ci: add workflow for uploading installers to release assets (#208)

END_COMMIT_OVERRIDE

@AnqiPang AnqiPang requested a review from vsiravar February 4, 2023 00:04
Signed-off-by: Anqi Pang <[email protected]>
@AnqiPang AnqiPang merged commit ff8a35f into runfinch:main Feb 7, 2023
@AnqiPang AnqiPang deleted the anqi/upload-pkg branch February 7, 2023 18:05
@ningziwen
Copy link
Member

nit: should be this "ci" isntead of "feat"?

@AnqiPang
Copy link
Contributor Author

AnqiPang commented Feb 7, 2023

nit: should be this "ci" isntead of "feat"?

Oh sorry for the mistake. Do we have any process to rewrite the commit message on main branch?

@AnqiPang
Copy link
Contributor Author

AnqiPang commented Feb 7, 2023

I think we can fix the release notes at the next release. WDYT? @ningziwen

@sam-berning
Copy link
Contributor

You can change the commit message release-please uses with this process: https://github.com/googleapis/release-please#how-can-i-fix-release-notes

@AnqiPang AnqiPang changed the title feat: add workflow for uploading installers to release assets ci: add workflow for uploading installers to release assets Feb 7, 2023
@AnqiPang
Copy link
Contributor Author

AnqiPang commented Feb 7, 2023

Following this example PR, I updated the merged PRs #208 and #202 with overridden commit message. Need to validate the release note is fixed after the next release-please run.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants