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

Several improvements #8

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Several improvements #8

wants to merge 5 commits into from

Conversation

ianhi
Copy link
Collaborator

@ianhi ianhi commented Oct 27, 2021

  1. Automated git init and first comit
  2. More versioning schemes (default is now setuptools_scm
  3. added pre-commit config

@tlambert03 this draws heavily from your pyrepo-cookiecutter (and the pre-commit from my fork of that) is this acknowledgement in licenses good for you or could you suggest something better?

@lgtm-com
Copy link

lgtm-com bot commented Oct 27, 2021

This pull request introduces 3 alerts and fixes 1 when merging e298867 into 5d1b8d4 - view on LGTM.com

new alerts:

  • 1 for Constant in conditional expression or statement
  • 1 for Syntax error
  • 1 for Comparison of constants

fixed alerts:

  • 1 for Unused import

@tlambert03
Copy link

is this acknowledgement in licenses good for you or could you suggest something better?

👍 just fine thanks :)

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