Skip to content

Conversation

@Ludmila-N
Copy link
Collaborator

No description provided.

@Ludmila-N Ludmila-N requested review from gruetter, lenucksi and rrrutledge and removed request for gruetter December 23, 2019 01:32
@rrrutledge
Copy link
Contributor

Great! Will look next week.

Copy link
Member

@lenucksi lenucksi left a comment

Choose a reason for hiding this comment

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

Thanks for making this review, finding and fixing the broken parts. I've added a few extra typo fixes and a few comments one of which I think should definitely be addressed.

Co-Authored-By: Johannes Tigges <lenucksi@users.noreply.github.com>
@Ludmila-N Ludmila-N requested a review from a team as a code owner December 25, 2019 00:47
Ludmila-N and others added 2 commits December 24, 2019 19:48
Co-Authored-By: Johannes Tigges <lenucksi@users.noreply.github.com>
Co-Authored-By: Johannes Tigges <lenucksi@users.noreply.github.com>
Copy link
Contributor

@rrrutledge rrrutledge left a comment

Choose a reason for hiding this comment

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

LGTM. Improvements possible as well!

@Ludmila-N Ludmila-N requested a review from lenucksi January 5, 2020 20:18
Copy link
Contributor

@rrrutledge rrrutledge left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Member

@lenucksi lenucksi left a comment

Choose a reason for hiding this comment

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

Thanks for the changes, looks good to me now.

@Ludmila-N Ludmila-N merged commit 5048f15 into InnerSourceCommons:master Jan 6, 2020
@Ludmila-N Ludmila-N deleted the Style-benefits branch January 28, 2020 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants