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

Cleanup requirements, docker file #507

Merged
merged 5 commits into from
Apr 7, 2024
Merged

Cleanup requirements, docker file #507

merged 5 commits into from
Apr 7, 2024

Conversation

sumpfork
Copy link
Owner

  • get rid of requirements.in in favour of directly compiling pyproject.toml into requirements.txt
  • add dev-requirements.txt in the same way, based on the dev requirements in pyproject.toml
  • require a reasonable python version > 3.7
  • simplify the docker file to
    • not pip-compile anything (it shouldn't, those versions are pinned for a reason)
    • not try to handle fonts (we can't distribute them)
    • not run --help by default - behaviour should be the same as the normal script
    • update the README to show how to map a the local dir so you can actually get the output file and use local fonts

@sumpfork sumpfork self-assigned this Mar 23, 2024
dockerfile Outdated Show resolved Hide resolved
@sumpfork sumpfork merged commit 1ea1673 into master Apr 7, 2024
7 checks passed
@sumpfork sumpfork deleted the cleanup_reqs branch April 7, 2024 23:09
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