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

Chore make the release process simply #228

Merged
merged 13 commits into from
Feb 27, 2022

Conversation

GiliFaroEnv0
Copy link
Contributor

@GiliFaroEnv0 GiliFaroEnv0 commented Feb 22, 2022

Issue & Steps to Reproduce / Feature Request

Solution

Add auto-generate for docs
update README on of you need to release properly 😄
removed unused "release drafter"

name: Update generated docs
run: ./generate-docs.sh
-
name: Commit changes
Copy link
Contributor

Choose a reason for hiding this comment

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

will this always create a commit?
can we do it only when the docs have changed ?

Copy link
Contributor Author

@GiliFaroEnv0 GiliFaroEnv0 Feb 27, 2022

Choose a reason for hiding this comment

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

It will Only commit IF it has something to commit 😄

README.md Outdated

1. Create and push a tag **locally**, in semver format - `git tag v0.0.9 && git push origin --tags`
2. New release with binaries **will be automatically generated** by the GitHub action defined in `.github/workflows/release.yml`.
2. The Registry will automatically pick up on the new version. If the registry says the release doesn't have any binaries, check the result of the `release` Github action. You might need to [Resync](https://registry.terraform.io/providers/env0/env0/latest/settings?pollNotifications=true) the registry after the action finishes.
Copy link
Contributor

Choose a reason for hiding this comment

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

I thought the whole point of this task was so that we don't have to rely on manual re-sync, because only I have that button.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@@ -13,7 +13,7 @@ jobs:
- name: Install Go
uses: actions/setup-go@v2
with:
go-version: 1.16 # please change also in `release.yml` and `go.mod`
go-version: 1.16 # please change also in `release.yml`,`update-generated-docs.yml` and `go.mod`
Copy link
Contributor

Choose a reason for hiding this comment

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

BTW keeping this as a Github Secret would be an easy way to share a single version between all the files.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@GiliFaroEnv0
Copy link
Contributor Author

@GiliFaroEnv0 GiliFaroEnv0 merged commit 1aa9e32 into main Feb 27, 2022
@GiliFaroEnv0 GiliFaroEnv0 deleted the chore-make-the-release-process-simply branch February 27, 2022 13:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants