Skip to content
This repository has been archived by the owner on Jan 22, 2024. It is now read-only.

Add help command to Makefile #42

Merged
merged 2 commits into from
Mar 6, 2020
Merged

Conversation

0xd669
Copy link
Contributor

@0xd669 0xd669 commented Mar 6, 2020

After reading this article, I guessed help command would be helpful. I just added some comments for each command and add help command. make help would print out the following result:

$ make help
Usage: 

 init           install requirements
 check          check if everything's okay
 format         format files
 test           run tests
 coverage       run tests with coverage
 htmlcov        run tests with coverage and create coverage report HTML files
 requirements   update requirements.txt
 help           prints this help message

sean-ahn
sean-ahn previously approved these changes Mar 6, 2020
@0xd669
Copy link
Contributor Author

0xd669 commented Mar 6, 2020

Oops, I guess tests for Makefile has broken. I shall fix it.

Copy link
Contributor

@winterjung winterjung left a comment

Choose a reason for hiding this comment

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

Thank you for contribution! Please make tests pass

Copy link
Contributor

@winterjung winterjung left a comment

Choose a reason for hiding this comment

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

LGTM

@0xd669 0xd669 merged commit b738bca into master Mar 6, 2020
@0xd669 0xd669 deleted the add-help-command-to-makefile branch March 6, 2020 05:54
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Development

Successfully merging this pull request may close these issues.

3 participants