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

Default manifest #855

Merged
merged 16 commits into from
Dec 5, 2024
Merged

Default manifest #855

merged 16 commits into from
Dec 5, 2024

Conversation

oscarlevin
Copy link
Member

This is the start of work to make it possible for a user to build a standalone pretext document. So far it just creates a default project when no manifest is found in the tree above the cwd. So if someone has their directory set up as default but doesn't have project.ptx, things will still work.

The next step (before this gets merged) will be to have the CLI build and view commands accept an optional additional argument for a filename that will be used as the source.

Note: I ran into some issues with Python 3.13 and mypy, complaining about Path objects not being allowed in "contexts" There were a couple spots where we were doing this, so I converted those. In one case that meant using a file_like object instead (with open(Path, "r") as file) and in the other just declaring the needed variable instead of using a context.

@StevenClontz, feel free to weigh in at any time if you like, but no pressure.

@oscarlevin oscarlevin force-pushed the default-manifest branch 2 times, most recently from a4d4104 to 2a4fc48 Compare November 11, 2024 16:54
@oscarlevin oscarlevin merged commit 3835f41 into main Dec 5, 2024
8 checks passed
@oscarlevin oscarlevin deleted the default-manifest branch December 5, 2024 23:59
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.

1 participant