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

Automatic setup/cleanup in Makefile. #183

Merged
merged 1 commit into from
Nov 22, 2021

Conversation

Lunderberg
Copy link
Contributor

A couple of setup items that I ran into while attempting to run make format for #177. Since I ran into them by foolishly skipping past the README, I assume that others might as well and added the following checks.

  • Automatically set up poetry environment when needed (e.g. for make test or make format). Can still be explicitly run using make setup.

  • Added make clean to remove the poetry environment.

  • Added make require, run during setup, to check environment for the presence of python3/poetry, and the python3 version. If any checks fail, show appropriate diagnostic messages.

- Automatically set up poetry environment when needed.  Can still be
  explicitly run using `make setup`.

- Added `make clean` to remove the poetry environment.

- Added `make require`, run during setup, to check environment for the
  presence of python3/poetry, and the python3 version.
Copy link
Owner

@pappasam pappasam left a comment

Choose a reason for hiding this comment

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

Wow, I love this! 👏 for helping improve the user experience for all contributors 🎉

@pappasam pappasam merged commit 8fe8340 into pappasam:main Nov 22, 2021
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.

2 participants