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

v0.21.1 - I've Made A Huge Mistake #1434

Merged
merged 5 commits into from
Jul 29, 2016
Merged

Conversation

shiftkey
Copy link
Member

@shiftkey shiftkey commented Jul 28, 2016

I published up v0.21 today without realising I missed calling out some of the new work that @maddin2016 had contributed. This is the remedy for that (yes, it's not technically a bugfix release but it'll help me sleep at night).

  • release notes
  • bump version
  • 👍 on release notes
  • :shipit:

@ryangribble
Copy link
Contributor

Oh dear! I saw it mentioned 0.21 was shipping tomorrow so I didn't review it earlier (that was a task for tonight) :grinning

I'll check these amendments out soon


**Features**

Due to an programming error in the tool to generate these release notes, additional
Copy link
Contributor

Choose a reason for hiding this comment

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

an programming => a programming

@ryangribble
Copy link
Contributor

I've run my own "PR's merged between 2 refs" code against this release and found another couple of PRs that aren't currently in the list:

PR#1398 Updated git data commit response with signature verification object via @Sarmad93

PR#1402 Fix serialization of enum value attributes via @maddin2016

PR#1341 Include reactions on issue comments via @alfhenrik

Due to an programming error in the tool to generate these release notes, additional
features were not properly documented for the previous release:

- Reactions preview API support for issues, commit comments and PR comments - #1335, #1405 via @maddin2016
Copy link
Contributor

Choose a reason for hiding this comment

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

There are actually 4 (not 3) items that Reactions are supported for: commit comments, issues, issue comments, pull request review comments

@shiftkey
Copy link
Member Author

@ryangribble thanks for the sanity check. I called out #1402 in the previous release, and I'll add the others here.

@ryangribble
Copy link
Contributor

ryangribble commented Jul 28, 2016

Weird, I'll need to look at why 1402 comes up in my query, 😀

I was confused when you said "previous release" (I thought you mean v0.20) so all good.

My query was from v0.20.0...master so 1402 coming up is correct, I just failed when seeing what you'd already listed in the release notes 😀

@shiftkey shiftkey force-pushed the now-with-added-release-notes branch from 0223b7a to eeca298 Compare July 29, 2016 00:53
@shiftkey
Copy link
Member Author

This is shipped (again)

@shiftkey shiftkey merged commit 40a60d7 into master Jul 29, 2016
@shiftkey shiftkey deleted the now-with-added-release-notes branch July 29, 2016 01:22
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