-
Notifications
You must be signed in to change notification settings - Fork 9
feat: improve template based on internal review #36
Conversation
f056c91
to
d9c2794
Compare
2be4b76
to
de51c6b
Compare
@@ -24,4 +24,4 @@ FROM $PYTHON_BASE | |||
COPY --from=deploy_builder /opt/venv /opt/venv | |||
|
|||
ENV PATH="/opt/venv/bin:$PATH" | |||
ENTRYPOINT [""] | |||
ENTRYPOINT ["{{ cookiecutter.project_name }}"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
name is slug? my-tool or my_tool ?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
slug=my_tool
project_name=my-tool
@@ -52,12 +52,14 @@ setup: setup-checks ## install dev requirements | |||
virtualenv --python=$(PYTHONVERSION) $(PYTHONENV) | |||
$(ENVPIP) install -r requirements_dev.txt | |||
|
|||
clean-all: clean |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think clean should always be clean all.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can't be because otherwise some commands will fail due to missing virtualenv (eg. dist target)
"use_spellcheck": "y", | ||
"use_spellcheck": "n", | ||
|
||
"use_tox": "n", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No Tox by default?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nope, it is still an experimental feature (same as Makefile)
This PR contains various changes/improvements which are: