Skip to content

Conversation

@Simran-B
Copy link
Contributor

@Simran-B Simran-B commented Dec 4, 2018

No description provided.

@Simran-B Simran-B self-assigned this Dec 4, 2018
@Simran-B Simran-B requested a review from hkernbach December 4, 2018 12:54
@Simran-B Simran-B requested review from maxkernbach and removed request for hkernbach December 4, 2018 17:17
Copy link
Contributor

@maxkernbach maxkernbach left a comment

Choose a reason for hiding this comment

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

LGTM

Did you use a script to format the text/code sections?

@Simran-B
Copy link
Contributor Author

Simran-B commented Dec 5, 2018

@maxkernbach: No, I tried to re-format something with a regex, but it didn't work as expected in vscode. I ended up doing it manually and case by case (I checked the current online version to see where code examples were too wide = scrollbars and fixed these)

Copy link
Contributor

@sleto-it sleto-it left a comment

Choose a reason for hiding this comment

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

Please do NOT change any code in a doc PR unless strictly strictly needed (it was not in this case). Rather open different PR with the code changes

@sleto-it sleto-it merged commit f0b7625 into master Dec 5, 2018
@BCaxelbecker BCaxelbecker deleted the documentation/2018-12-04 branch January 30, 2019 21:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants