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

refactor(ci): improve the release workflow #303

Merged
merged 1 commit into from
Sep 1, 2022
Merged

Conversation

behnazh
Copy link
Collaborator

@behnazh behnazh commented Aug 31, 2022

Closes #302 #301

@behnazh behnazh requested a review from jenstroeger August 31, 2022 23:42
Copy link
Owner

@jenstroeger jenstroeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh nice, thank you @behnazh! You know, I wonder if we can utilize the commitizen-action 🤔

And to be sure, on PR the build.yaml also builds and stores the artifacts with the workflow, right?

.github/workflows/build.yaml Outdated Show resolved Hide resolved
@behnazh
Copy link
Collaborator Author

behnazh commented Sep 1, 2022

I wonder if we can utilize the commitizen-action thinking

I thought we wanted to avoid using third-party GHAs when we switched from semantic release? Especially in this case we need to give write access, so I think it's better to remove such third-party GHAs.

@behnazh
Copy link
Collaborator Author

behnazh commented Sep 1, 2022

And to be sure, on PR the build.yaml also builds and stores the artifacts with the workflow, right?

Yes, see the artifacts here.

@behnazh behnazh force-pushed the improve-release-gha branch from 14c9c10 to 832d493 Compare September 1, 2022 03:30
Copy link
Owner

@jenstroeger jenstroeger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good 🤓

.github/workflows/release.yaml Show resolved Hide resolved
@behnazh behnazh merged commit c755a09 into staging Sep 1, 2022
@jenstroeger jenstroeger deleted the improve-release-gha branch September 1, 2022 07:02
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.

2 participants