Skip to content

v0.12#187

Merged
jpmelos merged 3 commits into
masterfrom
release-0.12
Mar 25, 2021
Merged

v0.12#187
jpmelos merged 3 commits into
masterfrom
release-0.12

Conversation

@jpmelos

@jpmelos jpmelos commented Mar 24, 2021

Copy link
Copy Markdown
Member

Going to release 0.12 before merging #185 and #186.

@jpmelos jpmelos requested a review from thomasst March 24, 2021 11:54
@jpmelos jpmelos self-assigned this Mar 24, 2021
@jpmelos

jpmelos commented Mar 24, 2021

Copy link
Copy Markdown
Member Author

@thomasst any PRs you'd like to merge before we release 0.12? I see you have 3 PRs recently opened.

@thomasst

Copy link
Copy Markdown
Member

There is another fix I wanted to do today or tomorrow. I'll see if I can squeeze it in today.

@jpmelos

jpmelos commented Mar 24, 2021

Copy link
Copy Markdown
Member Author

No worries, I can wait.

@thomasst

Copy link
Copy Markdown
Member

FYI, this one is still pending a second review: #188

@thomasst thomasst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should be ready to release now. Can you amend the changelog?

@jpmelos jpmelos requested a review from thomasst March 25, 2021 14:11

@thomasst thomasst left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Looks good. In the future I would probably just refer to the shortened commit hash in each change log entry to avoid linking to GitHub-specific stuff. No big deal though.

@jpmelos jpmelos merged commit 97d7642 into master Mar 25, 2021
@jpmelos jpmelos deleted the release-0.12 branch March 25, 2021 17:31
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