Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Commit

Permalink
Specify deps in pyproject.toml
Browse files Browse the repository at this point in the history
- Nuke python_dependencies and setup.py
- Allow commiting poetry.lock
- Initial lockfile pulling in the latest version of deps at the time of locking

Use newest signedjson

This includes a community PR which only requires importlib_metadata when
it's needed (on Python 3.7). This might alleviate some poetry pain.
  • Loading branch information
David Robertson committed Mar 30, 2022
1 parent 8b22534 commit fad21e0
Show file tree
Hide file tree
Showing 10 changed files with 3,034 additions and 346 deletions.
2 changes: 0 additions & 2 deletions .dockerignore
Original file line number Diff line number Diff line change
Expand Up @@ -4,8 +4,6 @@
# things to include
!docker
!synapse
!MANIFEST.in
!README.rst
!setup.py

**/__pycache__
4 changes: 4 additions & 0 deletions .gitignore
Original file line number Diff line number Diff line change
Expand Up @@ -15,6 +15,10 @@ _trial_temp*/
.DS_Store
__pycache__/

# We do want the poetry lockfile. TODO: is there a good reason for ignoring
# '*.lock' above? If not, let's nuke it.
!poetry.lock

# stuff that is likely to exist when you run a server locally
/*.db
/*.log
Expand Down
2 changes: 1 addition & 1 deletion docker/Dockerfile
Original file line number Diff line number Diff line change
Expand Up @@ -46,7 +46,7 @@ RUN \
&& rm -rf /var/lib/apt/lists/*

# Copy just what we need to pip install
COPY MANIFEST.in README.rst setup.py /synapse/
COPY MANIFEST.in README.rst /synapse/
COPY synapse/__init__.py /synapse/synapse/__init__.py
COPY synapse/python_dependencies.py /synapse/synapse/python_dependencies.py

Expand Down
1 change: 0 additions & 1 deletion mypy.ini
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ no_implicit_optional = True

files =
scripts-dev/,
setup.py,
synapse/,
tests/

Expand Down
Loading

0 comments on commit fad21e0

Please sign in to comment.