Skip to content

refactor: Rename a function and add todo comments#1375

Merged
gr2m merged 2 commits intobetafrom
coverage-todos
Jan 14, 2019
Merged

refactor: Rename a function and add todo comments#1375
gr2m merged 2 commits intobetafrom
coverage-todos

Conversation

@paulmelnikow
Copy link
Copy Markdown
Member

@paulmelnikow paulmelnikow commented Jan 12, 2019

This is the result of live triage with @gr2m and @jlilja: #1374 (comment)

@paulmelnikow
Copy link
Copy Markdown
Member Author

Hey @gr2m what do you think about merging this? We can add on the rest in a second PR on Wednesday.

Copy link
Copy Markdown
Member

@gr2m gr2m left a comment

Choose a reason for hiding this comment

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

Agreed

@gr2m gr2m merged commit abc1beb into beta Jan 14, 2019
@gr2m gr2m deleted the coverage-todos branch January 14, 2019 18:17
@nockbot
Copy link
Copy Markdown
Collaborator

nockbot commented Jan 19, 2019

🎉 This PR is included in version 11.0.0-beta.4 🎉

The release is available on:

Your semantic-release bot 📦🚀

@nockbot
Copy link
Copy Markdown
Collaborator

nockbot commented Aug 13, 2019

🎉 This PR is included in version 11.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants