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 thru october #34731

Merged
merged 5 commits into from
Oct 15, 2019

Conversation

kevingranade
Copy link
Member

Summary

SUMMARY: None

Purpose of change

Logs for the change god!
Turns out there was an issue where a certain configuration of PRs would make the changelog generation script give up early and not match PRs to commits.

Describe the solution

Make the changelog generation script not prune un-merged PRs out of it's fetch routine. This means the script fetches more stuff, but insure it gets everything it needs.
Reverted the previously comitted July and August changelogs because some of them were tainted by this bug.
Then the usual import followed by copyediting.

@ZhilkinSerg ZhilkinSerg added <Documentation> Design documents, internal info, guides and help. Other Things that can't be classified anywhere else labels Oct 14, 2019
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
data/changelog.txt Outdated Show resolved Hide resolved
@aoae23
Copy link
Contributor

aoae23 commented Oct 14, 2019

It's beautiful...

Add some periods.

Co-Authored-By: ZhilkinSerg <[email protected]>
@ZhilkinSerg ZhilkinSerg merged commit ecfca46 into CleverRaven:master Oct 15, 2019
Ramza13 pushed a commit to Ramza13/Cataclysm-DDA that referenced this pull request Oct 22, 2019
* Revert "July and August changelogs (CleverRaven#34095)"

This reverts commit 7a20bb6.

* Have changelog generation script consider unmerged PRs too

* Import changelog entries through 10-09

* Copyedit changelog

* Apply suggestions from code review

Add some periods.

Co-Authored-By: ZhilkinSerg <[email protected]>
Ramza13 pushed a commit to Ramza13/Cataclysm-DDA that referenced this pull request Nov 5, 2019
* Revert "July and August changelogs (CleverRaven#34095)"

This reverts commit 7a20bb6.

* Have changelog generation script consider unmerged PRs too

* Import changelog entries through 10-09

* Copyedit changelog

* Apply suggestions from code review

Add some periods.

Co-Authored-By: ZhilkinSerg <[email protected]>
Ramza13 pushed a commit to Ramza13/Cataclysm-DDA that referenced this pull request Nov 5, 2019
* Revert "July and August changelogs (CleverRaven#34095)"

This reverts commit 7a20bb6.

* Have changelog generation script consider unmerged PRs too

* Import changelog entries through 10-09

* Copyedit changelog

* Apply suggestions from code review

Add some periods.

Co-Authored-By: ZhilkinSerg <[email protected]>
Ramza13 pushed a commit to Ramza13/Cataclysm-DDA that referenced this pull request Nov 5, 2019
* Revert "July and August changelogs (CleverRaven#34095)"

This reverts commit 7a20bb6.

* Have changelog generation script consider unmerged PRs too

* Import changelog entries through 10-09

* Copyedit changelog

* Apply suggestions from code review

Add some periods.

Co-Authored-By: ZhilkinSerg <[email protected]>
@kevingranade kevingranade deleted the changelog-thru-october branch December 16, 2019 22:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
<Documentation> Design documents, internal info, guides and help. Other Things that can't be classified anywhere else
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants