Skip to content

Change default value for cleanup to False in latex_cleanup #251

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

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Change default value for cleanup to False in latex_cleanup #251

Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
db3dd9b
Change default value for cleanup to False
cmarmo Jul 4, 2025
f04420a
Modify configuration default value, improve documentation
cmarmo Aug 1, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion jupyterlab_latex/build.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@
from .util import run_command

@contextmanager
def latex_cleanup(cleanup=True, workdir='.', whitelist=None, greylist=None):
def latex_cleanup(cleanup=False, workdir='.', whitelist=None, greylist=None):
Copy link
Member

Choose a reason for hiding this comment

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

Does it work? It would seem this default is not used in:

jupyterlab-latex/jupyterlab_latex/build.py

Lines 306 to 307 in db3dd9b

with latex_cleanup(
cleanup=c.cleanup,

Can it be changed on the trait level?

jupyterlab-latex/jupyterlab_latex/config.py

Lines 29 to 30 in 58abc9c

cleanup = Bool(default_value=True, config=True,
help='Whether to clean up ".out/.aux" files or not.')

Copy link
Member

Choose a reason for hiding this comment

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

Also I guess the help message should clarify that it is more aggressive than just out/aux

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Does it work? It would seem this default is not used in:

Right! Sorry I didn't catch how it's working.
I have modified config.py and clarified (I hope) the documentation.
Thanks for your review!

"""Context manager for changing directory and removing files when done.

By default it works in the current directory, and removes all files that
Expand Down