Skip to content

REL: 4.8.0#472

Closed
jvkersch wants to merge 3 commits into
masterfrom
rel/4.8.0-second-try
Closed

REL: 4.8.0#472
jvkersch wants to merge 3 commits into
masterfrom
rel/4.8.0-second-try

Conversation

@jvkersch
Copy link
Copy Markdown
Contributor

Picks up from #460 and adds three more sets of changes (#469, #470, #471)

@mdickinson Since you already reviewed the earlier release PR, do you want to have another look here?

@corranwebster
Copy link
Copy Markdown
Contributor

LGTM

@mdickinson
Copy link
Copy Markdown
Member

I hope this wasn't still waiting on me :-(. I'll do some testing this afternoon, but the release needn't wait on that.

@jvkersch
Copy link
Copy Markdown
Contributor Author

jvkersch commented Aug 1, 2019

@mdickinson I'm embarrassed to say that when I created the release PR I managed to not include your fixes. I need to revisit that and create a 4.8.0-third-times-the-charm release branch...

@mdickinson
Copy link
Copy Markdown
Member

@jvkersch Okay, will wait on that. Please ping me on the new PR when it happens.

@jvkersch jvkersch mentioned this pull request Aug 4, 2019
@jvkersch
Copy link
Copy Markdown
Contributor Author

jvkersch commented Aug 4, 2019

Replaced by #476

@jvkersch jvkersch closed this Aug 4, 2019
@jvkersch jvkersch deleted the rel/4.8.0-second-try branch August 4, 2019 10:23
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