Merge pull request #754 from consideRatio/pr/remove-requirements-txt-fixer

pre-commit: remove requirements-txt-fixer
This commit is contained in:
Erik Sundell
2021-11-03 22:08:55 +01:00
committed by GitHub
2 changed files with 4 additions and 6 deletions

View File

@@ -40,9 +40,6 @@ repos:
# Autoformat: Makes sure files end in a newline and only a newline.
- id: end-of-file-fixer
# Autoformat: Sorts entries in requirements.txt.
- id: requirements-txt-fixer
# Lint: Check for files with names that would conflict on a
# case-insensitive filesystem like MacOS HFS+ or Windows FAT.
- id: check-case-conflict

View File

@@ -1,9 +1,10 @@
# When tljh.installer runs, the users' environment as typically found in
# /opt/tljh/user, is setup with these packages.
#
# FIXME: a frozen version of this file should be used
# pinning only direct dependencies is a recipe for broken environments!
# WARNING: The order of these dependencies matters, this was observed when using
# the requirements-txt-fixer pre-commit hook that sorted them and made
# our integration tests fail.
#
# JupyterHub + notebook package are base requirements for user environment
jupyterhub==1.*
notebook==6.*