-
-
Notifications
You must be signed in to change notification settings - Fork 44
Support Jupytext #745
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
Support Jupytext #745
Conversation
d6b0b68
to
7d013fb
Compare
Fairly confident about this - I've tried it out on a few projects, and haven't noticed any issues |
Just noticed this from pymc-examples:
probably a blocker EDIT: resolved, reported to jupytext mwouts/jupytext#994 |
Another blocker: -kernelspec:
- name: python3
- display_name: python3
jupytext:
text_representation:
extension: .md
format_name: myst
- format_version: '0.13'
- jupytext_version: 1.13.8
-execution:
- timeout: 300
+ format_version: 0.13
+ jupytext_version: 1.14.1
+kernelspec:
+ display_name: python3
+ name: python3 maybe, should just...not overwrite the header? not sure if:
gonna go with the former, but will try to get around it where possible. check mwouts/jupytext#656 for the execution one |
I've tried this out on a bunch of repos, and added tests, and it seems to work fine Tempted to just merge, more issues will probably only be discovered if this gets out and people start using it |
Is support for |
This is now possible after nbQA-dev/nbQA#745 which solved this issue (nbQA-dev/nbQA#668) I opened a year ago.
This is now possible after nbQA-dev/nbQA#745 which solved this issue (nbQA-dev/nbQA#668) I opened a year ago.
I removed it from the docs, but you can still use it |
cc @OriolAbril as you're using jupytext, in case this is useful to you |
* Add NBQA for notebook and docs linting This is now possible after nbQA-dev/nbQA#745 which solved this issue (nbQA-dev/nbQA#668) I opened a year ago. * Run pre-commit filters on all files * Lint * bump * [pre-commit.ci] auto fixes from pre-commit.com hooks for more information, see https://pre-commit.ci * update os * Fix all nbqa issues --------- Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
closes #668