-
Notifications
You must be signed in to change notification settings - Fork 14
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
Prevent changing kernel spec name in notebooks #22
Comments
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
Added initial check script that detected these current issues:
|
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
fixed the wrong kernel specs and added the check as pre-commit hook |
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
soxofaan
added a commit
that referenced
this issue
Jul 31, 2023
added github actions workflow now too |
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
eec1fba added note in README to not change the kernel spec of a notebook (because that messes up the end-user experience).
I think we should enforce this better with a check script and github action
The text was updated successfully, but these errors were encountered: