Skip to content

Preparing release version 5.2.4 #6198

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
Nov 15, 2019
Merged

Preparing release version 5.2.4 #6198

merged 1 commit into from
Nov 15, 2019

Conversation

asottile
Copy link
Member

No description provided.

* Bruno Oliveira
* Daniel Hahler
* Hugo
* Michael Shields
Copy link
Contributor

@blueyed blueyed Nov 15, 2019

Choose a reason for hiding this comment

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

Seems like this list is off? (nitpick, not relevant here)

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah no idea how towncrier computes this -- perhaps it includes reporters of associated issues?

Copy link
Member

Choose a reason for hiding this comment

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

It is actually computed by release.py:

# Get our list of authors
stdout = check_output(["git", "describe", "--abbrev=0", "--tags"])
stdout = stdout.decode("utf-8")
last_version = stdout.strip()
stdout = check_output(
["git", "log", "{}..HEAD".format(last_version), "--format=%aN"]
)
stdout = stdout.decode("utf-8")
contributors = set(stdout.splitlines())

@asottile please make sure you have the tags locally.

Copy link
Member Author

Choose a reason for hiding this comment

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

I do have them -- here's what it reports:

$ git describe --abbrev=0 --tags
5.2.3
$ git log 5.2.3..HEAD --format=%aN
Anthony Sottile
Anthony Sottile
Anthony Sottile
Anthony Sottile
Bruno Oliveira
Daniel Hahler
Bruno Oliveira
Michael Shields
Hugo
Hugo

which is weird given:

$ git log --graph --oneline --decorate
* c9a96cdee (HEAD -> release-5.2.4, asottile/release-5.2.4) Preparing release version 5.2.4
*   19a15a94e (origin/master, origin/HEAD, master) Merge pull request #6197 from asottile/fix_init_py_discovery
|\  
| * 4e0f99260 (asottile/fix_init_py_discovery, fix_init_py_discovery) Add regression tests for __init__.py breakage
| * 176c7771f Revert "fix bug with nonskipped first test in package (#5831)"
|/  
*   e856638ba Preparing release version 5.2.3 (#6190)
|\  
| * dd9a27cf5 (tag: 5.2.3) Adjust CHANGELOG

Copy link
Contributor

Choose a reason for hiding this comment

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

The new tag is 5.2.4.

Copy link
Member Author

Choose a reason for hiding this comment

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

yes, this command is "what has changed since the last tag"

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah.. that's because the tag is not on the master branch I think.

@asottile asottile merged commit 3a668ea into pytest-dev:master Nov 15, 2019
@asottile asottile deleted the release-5.2.4 branch November 15, 2019 22:57
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