Skip to content

Echoing out project name#9

Merged
sweeney merged 1 commit intohraban:masterfrom
olivercaine:master
Jan 3, 2018
Merged

Echoing out project name#9
sweeney merged 1 commit intohraban:masterfrom
olivercaine:master

Conversation

@olivercaine
Copy link
Copy Markdown
Contributor

Echoing out project name to help locate issues (especially useful when converting many projects into a mono repo takes a long time)

…n converting many projects into a mono repo takes a long time)
@hraban
Copy link
Copy Markdown
Owner

hraban commented Nov 8, 2017

Thanks for the PR :)

I'm not sure how useful it is to have an extra debug printf right after the one two lines up. Maybe you could change that to include the project name?

(additionally: cosmetic, but if possible please keep commit messages in git commit format. 50 lines summary, empty line, elaboration in 72 max width. this plays nice with GitHub and other tools. see https://chris.beams.io/posts/git-commit/ )

@sweeney
Copy link
Copy Markdown
Contributor

sweeney commented Jan 3, 2018

LGTM

@sweeney sweeney merged commit ec59cb0 into hraban:master Jan 3, 2018
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