Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add make help message and update contributing instructions #249

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

tdstein
Copy link
Collaborator

@tdstein tdstein commented Jul 29, 2024

No description provided.

@tdstein tdstein requested review from jonkeane and toph-allen July 29, 2024 14:40
Copy link

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
1227 1158 94% 0% 🟢

New Files

No new covered files...

Modified Files

No covered modified files...

updated for commit: 36de770 by action🐍

Comment on lines +64 to +81
help:
@echo "Makefile Targets"
@echo " all Run deps, dev, test, lint, and build"
@echo " build Build the project"
@echo " clean Clean up project artifacts"
@echo " cov Generate a coverage report"
@echo " cov-html Generate an HTML coverage report and open it"
@echo " cov-xml Generate an XML coverage report"
@echo " deps Install dependencies"
@echo " dev Install the project in editable mode"
@echo " docs Build the documentation"
@echo " fmt Format the code"
@echo " install Install the built project"
@echo " it Run integration tests"
@echo " lint Lint the code"
@echo " test Run unit tests with coverage"
@echo " uninstall Uninstall the project"
@echo " version Display the project version"
Copy link
Collaborator

Choose a reason for hiding this comment

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

OH, this is nice + helpful. I will say that it will really cut out my RTFSing of makefile time though!

@tdstein tdstein merged commit 7471a26 into main Jul 30, 2024
30 checks passed
@tdstein tdstein deleted the tdstein/update-readme branch July 30, 2024 13:31
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