-
Notifications
You must be signed in to change notification settings - Fork 8
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: release from release branch #147
Conversation
Updates the `cd.yaml` job so that we don't do a release on every commit. Introduces a new manual release job that we can use to decide when to do a release. Releases will come from the `release` branch rather than the `main` 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.
Nice LGTM! 🚢
- name: Build | ||
run: | | ||
dotnet build | ||
shell: bash | ||
|
||
- name: Unit Test | ||
run: | | ||
dotnet test tests/Unit/Momento.Sdk.Tests | ||
shell: bash | ||
|
||
- name: Integration Test | ||
run: | | ||
dotnet test tests/Integration/Momento.Sdk.Tests | ||
shell: bash | ||
|
||
- name: Incubating Integration Test | ||
run: | | ||
dotnet test tests/Integration/Momento.Sdk.Incubating.Tests | ||
shell: bash |
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.
Not a blocker but is there an easy way in GH actions to dry duplicated steps, eg this block vs what's in ci.yaml
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.
it can be done. pete did some of it in the examples repo, see the .github/actions
dir. I also have some re-usable actions in standards-and-practices
.
Updates the
cd.yaml
job so that we don't do a release onevery commit. Introduces a new manual release job that we
can use to decide when to do a release. Releases will come
from the
release
branch rather than themain
branch.