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

Docs on how to package a release + CHANGELOG for 0.24.0 #4697

Merged
merged 3 commits into from
Mar 27, 2018

Conversation

mistercrunch
Copy link
Member

No description provided.

@codecov-io
Copy link

codecov-io commented Mar 27, 2018

Codecov Report

Merging #4697 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #4697   +/-   ##
=======================================
  Coverage   71.52%   71.52%           
=======================================
  Files         190      190           
  Lines       14927    14927           
  Branches     1102     1102           
=======================================
  Hits        10677    10677           
  Misses       4247     4247           
  Partials        3        3

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update f9d85bd...47fc7ab. Read the comment docs.

CONTRIBUTING.md Outdated

# publish an update to the CHANGELOG.md for the right version range
# looking the latest CHANGELOG entry for the second argument
./gen_changelog.sh 0.25.0 0.22.1
Copy link
Contributor

Choose a reason for hiding this comment

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

versions looks reversed, also as an example something like 0.24 0.25 would be easier to follow

Copy link
Member Author

Choose a reason for hiding this comment

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

Following the logic in the underlying tool. I don't feel strongly about it either way. npm's github-changes

Copy link
Contributor

Choose a reason for hiding this comment

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

From gen_changelog.sh:

# usage:  ./github-changes 0.20.0 0.20.1
# will overwrites the local CHANGELOG.md, somehow you need to merge it in
github-changes -o apache -r incubator-superset --token $GITHUB_TOKEN --between-tags $1...$2

So $1 should be older than $2

Copy link
Member Author

Choose a reason for hiding this comment

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

Oh you're right!

@mistercrunch mistercrunch merged commit deb2111 into apache:master Mar 27, 2018
@mistercrunch mistercrunch deleted the CHANGELOG_0_24_0 branch March 27, 2018 22:52
john-bodley pushed a commit to john-bodley/superset that referenced this pull request Apr 10, 2018
* Docs on how to package a release + CHANGELOG for 0.24.0

* Addressing comments

* Add pypi perm info

(cherry picked from commit deb2111)
michellethomas pushed a commit to michellethomas/panoramix that referenced this pull request May 24, 2018
* Docs on how to package a release + CHANGELOG for 0.24.0

* Addressing comments

* Add pypi perm info
wenchma pushed a commit to wenchma/incubator-superset that referenced this pull request Nov 16, 2018
* Docs on how to package a release + CHANGELOG for 0.24.0

* Addressing comments

* Add pypi perm info
@mistercrunch mistercrunch added 🏷️ bot A label used by `supersetbot` to keep track of which PR where auto-tagged with release labels 🚢 0.25.0 labels Feb 27, 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 🚢 0.25.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants