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

feat: Add jupytext support for notebooks #7

Merged
merged 5 commits into from
Mar 16, 2024
Merged

Conversation

matthewfeickert
Copy link
Contributor

@matthewfeickert matthewfeickert commented Mar 16, 2024

Purpose

  • Add jupytext to requirements and rebuild lock files.
  • Add jupytext .ipynb to .py pairing rules to pyproject.toml.
  • Convert .ipynb to .py:percent with jupytext.
  • Ignore .ipynb files to make versioning easier.
  • Apply ruff formatting.
  • Add section on using jupytext to README.

Does this introduce a breaking change?

[x] Yes
[] No

Pull Request Type

What kind of change does this Pull Request introduce?

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[x] Other... Please describe:

@matthewfeickert matthewfeickert self-assigned this Mar 16, 2024
@matthewfeickert
Copy link
Contributor Author

I'm going to approve and merge this myself as other team members aren't available currently. As always, PRs approved by a single dev can be reverted as needed by the rest of the dev team.

@matthewfeickert matthewfeickert merged commit ec4cda7 into main Mar 16, 2024
1 of 2 checks passed
@matthewfeickert matthewfeickert deleted the feat/add-jupytext branch March 16, 2024 00:33
@guenp
Copy link
Member

guenp commented Mar 31, 2024

@matthewfeickert I didn't see this PR, could you tag me next time?

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.

2 participants