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

Update Offline Changelog #669

Merged
merged 2 commits into from
Oct 22, 2021
Merged

Update Offline Changelog #669

merged 2 commits into from
Oct 22, 2021

Conversation

github-actions[bot]
Copy link
Contributor

Automated changes by create-pull-request GitHub action

@netlify
Copy link

netlify bot commented Oct 19, 2021

✔️ Deploy Preview for k8gb-preview ready!

🔨 Explore the source changes: 5f5c006

🔍 Inspect the deploy log: https://app.netlify.com/sites/k8gb-preview/deploys/617009dbabf1b3000731a4a3

😎 Browse the preview: https://deploy-preview-669--k8gb-preview.netlify.app/changelog

CHANGELOG.md Outdated
@@ -1,5 +1,115 @@
# Changelog

## [Unreleased](https://github.com/k8gb-io/k8gb/tree/HEAD)
Copy link
Member

Choose a reason for hiding this comment

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

Do we need a v0.8.3 tag move?

Copy link
Member

Choose a reason for hiding this comment

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

Whether to show the "unreleased entries" can be controlled on the gh action level. This auto pr is opened from the action that is run against the tag vX.Y.Z -> release, however this action has failed (gh api limits) and meanwhile some commits/prs/issues have landed to master, then this action was re-run from gh web ui and it figured out there were some changes so it's showing them.

I think we should set the unreleased to false in order to suppress these entries. I have a PR opened that does also this. What do you mean by moving the tag? Delete and recreate on the tip of the master? That would trigger the new release, no? I'd rather tweak this pr and hide the unreleased section (in ad-hoc / manual fashion this time)

Copy link
Collaborator

Choose a reason for hiding this comment

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

I think this should be closed

Copy link
Member

Choose a reason for hiding this comment

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

@jkremser by moving tag i mean recreating the v0.8.3 release with the tip of the master yes.I am not sure if it is actually required in this specific case, so up to you to decide

@jkremser
Copy link
Member

@k0da why closing, we would lost the change log for 0.8.3 if it was closed? I've added 1 commit to it and now it looks good to me

@k0da
Copy link
Collaborator

k0da commented Oct 20, 2021

@jkremser right!. LGTM

Copy link
Member

@ytsarev ytsarev left a comment

Choose a reason for hiding this comment

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

LGTM, though would be good to understand why some lines end up in other sections as @somaritane pointed out :)

@ytsarev ytsarev requested a review from somaritane October 22, 2021 09:29
Copy link
Contributor

@somaritane somaritane left a comment

Choose a reason for hiding this comment

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

LGTM

@ytsarev ytsarev merged commit 07745c0 into master Oct 22, 2021
@ytsarev ytsarev deleted the offline_changelog branch October 22, 2021 09:37
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.

5 participants