search

flyingcircusio / appenv

Self-contained bootstrapping/updating of Python applications deployed through shared repositories
BSD 3-Clause "New" or "Revised" License
4 stars 1 forks source link

Introduce new command arg `prepare` #34

Closed sweh closed 1 year ago

sweh commented 2 years ago

This allows preparing the venv before running the app for the first time.

It allows installing the packages in a seperate layer (for better caching) when using AppEnv inside docker.

icemac commented 2 years ago

@sweh The linter complains about some newlines. And a unittest fails. Could you please have a look?

sweh commented 2 years ago

Hi icemac,

@sweh The linter complains about some newlines. And a unittest fails. Could you please have a look?

I was able to fix the linter issues. Unfortunately, I'm not able to run the Python3.6 tests on my development machine. I can't install Python3.6 because its not compatible with Apple Silicon. The tests under Python >= 3.7 are running fine.

icemac commented 2 years ago

@ctheune The tests are probably broken on master, too. At least test runs on master in the gocept fork show the same problem: https://github.com/gocept/appenv/runs/4440660809?check_suite_focus=true

ctheune commented 2 years ago

The typing-extensions issue is a typical issue where dependencies depend on the version of Python used. I'll have to look deeper here to help further.