Skip to content

Add Makefile generation during project creation#8

Merged
andrewnags merged 2 commits intomasterfrom
feat/make
Mar 2, 2021
Merged

Add Makefile generation during project creation#8
andrewnags merged 2 commits intomasterfrom
feat/make

Conversation

@andrewnags
Copy link
Copy Markdown
Collaborator

@andrewnags andrewnags commented Mar 1, 2021

When creating a project, generate a Makefile with targets for the

  • linter/formatter
  • type checker
  • test framework
    so that the user can use the same commands uniformly across frameworks.

For example, a user wouldn't need to know if a krait-generated
project used unittest, pytest, nose, etc. -- the user need only
type make test at the command prompt and the generated target will
perform the appropriate action.

Note that the user will have to upgrade with krait upgrade
for the required template files to be copied;
pip install -U krait won't work.

When creating a project, generate a Makefile with targets for the
- linter/formatter
- type checker
- test framework
so that the user can use the same commands uniformly across frameworks.

For example, a user wouldn't need to know if a krait-generated
project used `unittest`, `pytest`, `nose`, etc. -- the user need only
type `make test` at the command prompt and the generated target will
perform the appropriate action.
@andrewnags andrewnags requested a review from cryptaliagy as a code owner March 1, 2021 18:03
Copy link
Copy Markdown
Owner

@cryptaliagy cryptaliagy left a comment

Choose a reason for hiding this comment

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

lgtm, one small nit but otherwise feel free to merge

Comment thread src/krait/lib/plugins/base_plugin.py Outdated
Co-authored-by: Natalia Maximo <iam@natalia.dev>
@andrewnags andrewnags merged commit adce6ef into master Mar 2, 2021
@andrewnags andrewnags deleted the feat/make branch March 2, 2021 19:49
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