Skip to content

Release prep v8.2.1 #1362

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

Merged
merged 1 commit into from
Aug 24, 2022
Merged

Release prep v8.2.1 #1362

merged 1 commit into from
Aug 24, 2022

Conversation

github-actions[bot]
Copy link

@github-actions github-actions bot commented Aug 23, 2022

Automated release-prep through pdk-templates from commit b2e365a.
Please verify before merging:

@github-actions github-actions bot requested a review from a team as a code owner August 23, 2022 15:25
ekohl
ekohl previously requested changes Aug 23, 2022
Copy link
Collaborator

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

Please include #1363

@pmcmaw
Copy link
Contributor

pmcmaw commented Aug 24, 2022

#1363 has been included in the release.

@GSPatton GSPatton dismissed ekohl’s stale review August 24, 2022 09:07

change made

@@ -2,7 +2,17 @@

All notable changes to this project will be documented in this file. The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) and this project adheres to [Semantic Versioning](http://semver.org).

## [v8.2.0](https://github.com/puppetlabs/puppetlabs-postgresql/tree/v8.2.0) (2022-08-22)
## [v8.2.1](https://github.com/puppetlabs/puppetlabs-postgresql/tree/v8.2.1) (2022-08-23)
Copy link
Collaborator

Choose a reason for hiding this comment

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

Here is something I dislike about including the date in a file: it gets out of date quickly. Today it's already 2022-08-24, which you also saw with the 8.2.0 release: the changelog is changed again.

Copy link
Contributor

Choose a reason for hiding this comment

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

Yeah, I agree its not ideal. We aim to get our releases out on the day they have been prepped, but it is just not always possible. 👎

Copy link
Collaborator

Choose a reason for hiding this comment

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

We have the same thing in our modules. Someone from Europe creates a PR, later someone from the US reviews and the next day it's merged & released. Quite often things are messy and cleaning this up with automation is non-trivial :(

@GSPatton GSPatton merged commit 061f379 into main Aug 24, 2022
@GSPatton GSPatton deleted the release-prep branch August 24, 2022 09:07
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.

3 participants