Update the timestamp of the py file only if it is older than that of the nb file

[mwouts]

Closes #1215

[github-actions]

Thank you for making this pull request.

Did you know? You can try it on Binder: or .

Also, the version of Jupytext developed in this PR can be installed with pip:

HATCH_BUILD_HOOKS_ENABLE=true pip install git+https://github.com/mwouts/jupytext.git@skip_update_timestamp

(this requires nodejs, see more at Developing Jupytext)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.76%. Comparing base (7963f5f) to head (bd50cbe).

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1227      +/-   ##
==========================================
+ Coverage   97.73%   97.76%   +0.02%     
==========================================
  Files          29       29              
  Lines        4464     4468       +4     
==========================================
+ Hits         4363     4368       +5     
+ Misses        101      100       -1     

Flag	Coverage Δ
external	75.14% <71.42%> (-0.03%)	⬇️
functional	88.52% <100.00%> (+0.01%)	⬆️
integration	77.28% <85.71%> (-0.01%)	⬇️
unit	66.56% <85.71%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[max-accenta]

Hi @mwouts, sorry to bother you but after the recent change, this feature only works with the --sync command.

If I run jupytext --sync notebook.py I get:

[jupytext] Reading notebook.py in format py
[jupytext] Loading notebook.ipynb
[jupytext] Updating notebook.ipynb
[jupytext] Unchanged notebook.py

however with jupytext --to ipynb notebook.py --output notebook.ipynb I get:

[jupytext] Reading notebook.py in format py
[jupytext] Writing notebook.ipynb (destination file replaced [use --update to preserve cell outputs and ids])
[jupytext] Updating the timestamp of notebook.py

and the python file's timestamp is updated.

Is there a reason we can't get the same behavior for --to as for --sync ? Thanks