mwouts / jupytext

Jupyter Notebooks as Markdown Documents, Julia, Python or R scripts
https://jupytext.readthedocs.io
MIT License
6.65k stars 386 forks source link

ci: Minor reorganizations #1190

Closed LecrisUT closed 10 months ago

LecrisUT commented 11 months ago
github-actions[bot] commented 11 months ago

Thank you for making this pull request.

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

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

pip install git+https://github.com/LecrisUT/jupytext.git@ci/organize

(this requires nodejs, see more at Developing Jupytext)

codecov[bot] commented 11 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (0606f03) 97.73% compared to head (1870d19) 96.74%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #1190 +/- ## ========================================== - Coverage 97.73% 96.74% -0.99% ========================================== Files 29 29 Lines 4456 4456 ========================================== - Hits 4355 4311 -44 - Misses 101 145 +44 ``` | [Flag](https://app.codecov.io/gh/mwouts/jupytext/pull/1190/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marc+Wouts) | Coverage Δ | | |---|---|---| | [external](https://app.codecov.io/gh/mwouts/jupytext/pull/1190/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marc+Wouts) | `?` | | | [functional](https://app.codecov.io/gh/mwouts/jupytext/pull/1190/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marc+Wouts) | `?` | | | [integration](https://app.codecov.io/gh/mwouts/jupytext/pull/1190/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marc+Wouts) | `?` | | | [unit](https://app.codecov.io/gh/mwouts/jupytext/pull/1190/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marc+Wouts) | `?` | | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Marc+Wouts#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

LecrisUT commented 11 months ago

About the codecov issue (Unable to locate build via Github Actions API). Ping me if it happens again, and we can try setting env_vars ^1

mwouts commented 11 months ago

Hi @LecrisUT , thank you for making this PR! I have a busy week but I will certainly look into this over the coming days. Thanks!