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

fix: remove rando .git file #5

Merged
merged 1 commit into from
Jul 13, 2022
Merged

Conversation

vjeeva
Copy link
Contributor

@vjeeva vjeeva commented Jul 13, 2022

No description provided.

@vjeeva vjeeva merged commit 16d38fd into main Jul 13, 2022
vjeeva added a commit that referenced this pull request Jul 14, 2022
* fix: remove rando .git file (#5)

Co-authored-by: Varjitt Jeeva <varjitt.jeeva@autodesk.com>

* ci: update pre-commit config

This adds some new entries to pre-commit to help on this project. It
also adds a few supporting files:

* .prettierignore - like gitignore/dockerignore, but for prettier which
  formats files. This ignores all files in the .github folder
* .github/workflows/constraints.txt - This is a requirements.txt file
  for the base tools used in the repo. It defines pip, poetry, and
  virtualenv versions

The pre-commit runs the following checks:

* black - automatic code formatting
* check-added-large-files - Tries to prevent checking large
  files/binaries into git
* check-toml - Lints toml files (pyproject.toml)
* check-yaml - Lints yaml files
* end-of-file-fixer - Makes sure all files end in a new line
* flake8 - python style linting
* pyupgrade - automatically upgrades python syntax for newer versions of
  python. Currently set for python 3.7 and greater
* reoder-python-imports - sorts python imports in python files into
  groups
* trailing-whitespace - trims trailing whitespaces from lines
* prettier - generic file formatter

I added dev requirements to pyproject.toml for all of these pre-commits

Co-authored-by: Varjitt Jeeva <varjitt.jeeva@plangrid.com>
Co-authored-by: Varjitt Jeeva <varjitt.jeeva@autodesk.com>
@vjeeva vjeeva deleted the vjeeva/remove-git-random-files branch July 14, 2022 17:16
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.

1 participant