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

Changelog entries for 13.0.0 #7850

Open
wants to merge 2 commits into
base: release-13.0
Choose a base branch
from

Conversation

naisila
Copy link
Member

@naisila naisila commented Jan 17, 2025

No description provided.

Copy link
Member

@hanefi hanefi left a comment

Choose a reason for hiding this comment

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

A general guide on how we order changelog entries:

Order by lexicographical order first, then make adjustments when needed so that important items are at the beginning of the list.

e.g. a section of Adds .. followed by Allows .. followed by Propagates .. etc.

CHANGELOG.md Outdated

* Allows specifying an access method for distributed partitioned tables (#7818)

* Adds support for exclusion constraints on distributed partitioned tables (#7733)
Copy link
Member

Choose a reason for hiding this comment

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

We allow at most 80 characters in this changelog due to some limitations in Debian packaging infrastructure.

Suggested change
* Adds support for exclusion constraints on distributed partitioned tables (#7733)
* Adds support for exclusion constraints on distributed partitioned tables
(#7733)

CHANGELOG.md Outdated
Comment on lines 21 to 22
* Adds devcontainer support (#7739)

Copy link
Member

Choose a reason for hiding this comment

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

Devcontainer support is not a user facing feature, and we do not need to include that in the changelog.

Suggested change
* Adds devcontainer support (#7739)

CHANGELOG.md Outdated
@@ -1,3 +1,25 @@
### citus v13.0.0 (January 17, 2025) ###

* Adds support for PostgreSQL 17.2 (#7699, #7661)
Copy link
Member

Choose a reason for hiding this comment

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

Should we change this to say 17 instead?

Copy link

codecov bot commented Jan 17, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 89.50%. Comparing base (c55bc8c) to head (a26944c).
Report is 1 commits behind head on release-13.0.

Additional details and impacted files
@@               Coverage Diff                @@
##           release-13.0    #7850      +/-   ##
================================================
+ Coverage         89.48%   89.50%   +0.01%     
================================================
  Files               276      276              
  Lines             60063    60062       -1     
  Branches           7524     7523       -1     
================================================
+ Hits              53747    53756       +9     
+ Misses             4166     4159       -7     
+ Partials           2150     2147       -3     

@naisila
Copy link
Member Author

naisila commented Jan 17, 2025

@hanefi

Order by lexicographical order first, then make adjustments when needed so that important items are at the beginning of the list.

Got it. I tried to do that.

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