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: Splits the CHANGELOG into multiple files #26800

Merged
merged 2 commits into from
Jan 26, 2024

Conversation

michael-s-molina
Copy link
Member

SUMMARY

Splits the CHANGELOG.md into multiple files to better organize the contents of each release and also to deal with GitHub size limitation when displaying the file. Now every release will have its own file at CHANGELOG/<version>.md

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@michael-s-molina michael-s-molina requested a review from a team January 25, 2024 17:13
Copy link

codecov bot commented Jan 25, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (424b4c2) 67.16% compared to head (a2a51ed) 67.22%.
Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #26800      +/-   ##
==========================================
+ Coverage   67.16%   67.22%   +0.05%     
==========================================
  Files        1894     1894              
  Lines       74176    74178       +2     
  Branches     8243     8244       +1     
==========================================
+ Hits        49820    49864      +44     
+ Misses      22287    22242      -45     
- Partials     2069     2072       +3     
Flag Coverage Δ
javascript 56.97% <ø> (+0.11%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@john-bodley
Copy link
Member

Thanks @michael-s-molina for the cleanup. My only comment is whether there's merit in keeping the CHANGELOG.md which contains a list of links to the version specific change logs.

@michael-s-molina
Copy link
Member Author

Thanks @michael-s-molina for the cleanup. My only comment is whether there's merit in keeping the CHANGELOG.md which contains a list of links to the version specific change logs.

@john-bodley I applied your suggestion.

Apache {{ project_name }} version {{ version }} based on {{ version_rc }} is now closed.

{% if vote_negatives|length > 0 -%}
The vote did NOT PASS with {{vote_bindings|length}} binding +1, {{ vote_nonbindings|length}} non-binding +1, and {{vote_negatives|length}} -1 votes:
{% elif vote_bindings|length > 2 -%}
The vote PASSED with {{vote_bindings|length}} binding +1, {{ vote_nonbindings|length}} non-binding +1, and {{vote_negatives|length}} -1 votes:
{% else -%}
The vote is non conclusive with {{vote_bindings|length}} binding +1, {{ vote_nonbindings|length}} non-binding -1, and {{vote_negatives|length}} -1 votes:
The vote is non conclusive with {{vote_bindings|length}} binding +1, {{ vote_nonbindings|length}} non-binding +1, and {{vote_negatives|length}} -1 votes:
Copy link
Member

Choose a reason for hiding this comment

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

Nice catches. I just wonder if vote_nonbinding is either + OR -? I'm not seeing where this is tabulated to be sure. I'm guessing you're right though :D

Copy link
Member Author

Choose a reason for hiding this comment

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

It's defined here.

A List of people with +1 non binding vote (ex: Ville)

Copy link
Member

@rusackas rusackas left a comment

Choose a reason for hiding this comment

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

LGTM!!

Copy link
Member

@dpgaspar dpgaspar left a comment

Choose a reason for hiding this comment

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

Nice!

@michael-s-molina michael-s-molina merged commit 23460b2 into apache:master Jan 26, 2024
31 checks passed
eschutho pushed a commit to preset-io/superset that referenced this pull request Jan 31, 2024
sfirke pushed a commit to sfirke/superset that referenced this pull request Mar 22, 2024
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 4.0.0 labels Apr 17, 2024
vinothkumar66 pushed a commit to vinothkumar66/superset that referenced this pull request Nov 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels size/XXL 🚢 4.0.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants