Skip to content

put entry in CHANGELOG and release notes for 42.5.0#2601

Merged
davecramer merged 5 commits into
pgjdbc:masterfrom
davecramer:updatechangelog
Aug 24, 2022
Merged

put entry in CHANGELOG and release notes for 42.5.0#2601
davecramer merged 5 commits into
pgjdbc:masterfrom
davecramer:updatechangelog

Conversation

@davecramer

Copy link
Copy Markdown
Member

No description provided.

@davecramer davecramer changed the title put entry in CHANGELOG put entry in CHANGELOG and release notes for 42.5.0 Aug 23, 2022
@davecramer davecramer requested review from bokken, sehrope and vlsi August 23, 2022 18:56
Comment thread CHANGELOG.md

[42.5.0] (2022-08-23 11:20:11 -0400)
### Changed
- fix: revert change in [PR #1986](https://github.com/pgjdbc/pgjdbc/pull/1986) where float was aliased to float4 from float8.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Is the "-" prefix for the revert line intentional? Otherwise rest looks fine.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Yes, is intentional, when is rendered in markdown it looks like a

  • bullet point

Comment thread CHANGELOG.md Outdated

### Fixed

[42.5.0] (2022-08-23 11:20:11 -0400)

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Versions should start as a header with the prefix ##

## [42.5.0] (2022-08-23 11:20:11 -0400)

(42.4.1 and 42.4.2 have the same issue)

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

good catch, thx

Comment thread CHANGELOG.md Outdated

### Added

### Fixed

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If the sections Added and Fixed are empty we can remove them.

Comment thread CHANGELOG.md
[42.4.1]: https://github.com/pgjdbc/pgjdbc/compare/REL42.4.1...REL42.4.2
[Unreleased]: https://github.com/pgjdbc/pgjdbc/compare/REL42.4.2...HEAD
[42.4.2]: https://github.com/pgjdbc/pgjdbc/compare/REL42.4.2...REL42.5.0
[Unreleased]: https://github.com/pgjdbc/pgjdbc/compare/REL42.5.0...HEAD

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I have missed this in previous versions but the comparison order should be:

[VERSION]: .../compare/REL_PREVIOUS_VERSION...REL_VERSION (the final REL should match the linked version)

We have a duplicate version 42.3.4, and from there the versions are mixed.

Deleting all the lines since 42.3.4 (including the duplicate) and adding this should fix it:

[42.3.4]: https://github.com/pgjdbc/pgjdbc/compare/REL42.3.3...REL42.3.4
[42.3.5]: https://github.com/pgjdbc/pgjdbc/compare/REL42.3.4...REL42.3.5
[42.3.6]: https://github.com/pgjdbc/pgjdbc/compare/REL42.3.5...REL42.3.6
[42.4.0]: https://github.com/pgjdbc/pgjdbc/compare/REL42.3.6...REL42.4.0
[42.4.1]: https://github.com/pgjdbc/pgjdbc/compare/REL42.4.0...REL42.4.1
[42.4.2]: https://github.com/pgjdbc/pgjdbc/compare/REL42.4.1...REL42.4.2
[42.5.0]: https://github.com/pgjdbc/pgjdbc/compare/REL42.4.2...REL42.5.0
[Unreleased]: https://github.com/pgjdbc/pgjdbc/compare/REL42.5.0...HEAD

@davecramer

Copy link
Copy Markdown
Member Author

@jorsol looks good ?

I'd like to release this today if possible.

@davecramer

Copy link
Copy Markdown
Member Author

@vlsi @sehrope ? any issues with releasing ?

@davecramer davecramer merged commit f490edf into pgjdbc:master Aug 24, 2022
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.

3 participants