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

Add toml module as explicit requirement #4

Merged
merged 1 commit into from Jul 20, 2021
Merged

Conversation

mgmarino
Copy link
Contributor

black recently dropped this requirement
(psf/black#2301), so this results in an error if
this is run with a newer version of black.

black recently dropped this requirement
(psf/black#2301), so this results in an error if
this is run with a newer version of black.
@mgmarino
Copy link
Contributor Author

Note, we are currently working around this problem in our pre-commit yaml like:

repos:
-   repo: https://github.com/dfm/black_nbconvert
    rev: v0.2.0
    hooks:
    - id: black_nbconvert
      additional_dependencies: [toml]

@dfm dfm merged commit 40deaab into dfm:main Jul 20, 2021
@dfm
Copy link
Owner

dfm commented Jul 20, 2021

Thanks! I'll try to publish a release soon!

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.

None yet

2 participants