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.

Sign up for GitHub

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

Jump to bottom

Convert into main + sub packages instead of multiple main packages #76

Closed
wants to merge 29 commits into from
Closed

Convert into main + sub packages instead of multiple main packages #76

wants to merge 29 commits into from

Conversation

norbusan
Copy link
Collaborator

@norbusan norbusan commented Nov 28, 2024 •
edited
Loading

This PR creates a new package tex2pdf with the following sub-packages

  • tex2pdf.tex_inspection
  • tex2pdf.preflight_parser
  • tex2pdf.zerozeroreadme
  • tex2pdf.service

The pdf_profile package is left separate for now since it is only used during tests of genpdf.

See discussion here

@norbusan norbusan marked this pull request as draft November 28, 2024 06:51
@norbusan norbusan changed the title Convert into main + sub packages instead of multiple main packages [WIP] Convert into main + sub packages instead of multiple main packages Nov 28, 2024
bdc34
bdc34 previously approved these changes Dec 2, 2024
View reviewed changes
Copy link

@bdc34 bdc34 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good to me. I kind of like the idea of it being either a library or a service.

norbusan added 26 commits December 3, 2024 15:18
@norbusan
Move preflight into tex2pdf
f5d610d
@norbusan
Fix tests and mypy errors
196b48b
@norbusan
Cleanup after preflight_parser move
95f9f0e
@norbusan
Move zzrm into tex2pdf
3e1c25d
@norbusan
Fix typing and test running
75944a5
@norbusan
Move tex_inspection to tex2pdf
3910989
@norbusan
Fix preflight parser mypy test
fe235f3
@norbusan
Move tex2pdf_service to tex2pdf.service
c1288e2
@norbusan
Move remaining files of tex2pdf_service
3963e4c
@norbusan
Move remaining files of tex2pdf_service
e192aff
@norbusan
Missing __init__ files, pre-commit cleanup
c6732d4
@norbusan
reorganize imports
28ef6b7 
- drop pikepdf from pyproject and unused import
- make deps that are only used for the service an extra
@norbusan
add poetry.lock
4d0540c
@norbusan
Allow ruff-ing conftest.py
341b7ea
@norbusan
Fixes for missing packages and not-starting docker
dcb6f83
@norbusan
Fix docker tests for tex2pdf.service
1bfb178
@norbusan
Move preflight tests to test/preflight so they don't end up in the do…
2aaa3f3 
…cker image
@norbusan
Move zzrm tests to test/preflight so they don't end up in the docker …
5142fef 
…image
@norbusan
Move service tests to test/preflight so they don't end up in the dock…
ff69819 
…er image
@norbusan
Fixes for typing in tex2pdf.service
f21c209
@norbusan
Fixes for service unittests
49dd3a5
@norbusan
update poetry.lock
33c6d50
@norbusan
Test streamlining, all output in same dir
6c8f71e
@norbusan
Update README.md and authors
00ec1c5
@norbusan
Move tex_inspection tests into test directory
e54f420
@norbusan
Enable ruff for tests, fix failures, format
66bc48d
@norbusan norbusan changed the title [WIP] Convert into main + sub packages instead of multiple main packages Convert into main + sub packages instead of multiple main packages Dec 3, 2024
@norbusan norbusan marked this pull request as ready for review December 3, 2024 06:46
@bdc34 bdc34 self-requested a review December 3, 2024 14:30
bdc34
bdc34 previously approved these changes Dec 3, 2024
View reviewed changes
@bdc34 bdc34 self-requested a review December 10, 2024 21:30
bdc34
bdc34 previously approved these changes Dec 10, 2024
View reviewed changes
@norbusan
Copy link
Collaborator Author

Overridden by #77

@norbusan norbusan closed this Dec 19, 2024
@norbusan norbusan deleted the single-package branch December 19, 2024 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bdc34 bdc34 bdc34 left review comments

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@norbusan @bdc34