-
Notifications
You must be signed in to change notification settings - Fork 15
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
🪐 Thebe integration #21
Conversation
89758c8
to
3b66698
Compare
b636d28
to
0726db6
Compare
62bc3f7
to
bede18a
Compare
Wow, the Thebe support is super exciting. I can't wait to try out this new feature ✨ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Things I discovered in #51.
2c133ed
to
b66989c
Compare
* 🔧 Consume new thebe myst frontmatter * 🌎 Default to local server if present in config * 🤖 Use git modules for patches * 🐛 Types * Update packages --------- Co-authored-by: Rowan Cockett <rowanc1@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Going to move ahead with the merge and release!
"thebe-core": "^0.1.6", | ||
"thebe-react": "^0.0.7", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@stevejpurves I think it is worth pinning the thebe packages to the same release number?! This is how we are doing it in theme, and is a few less numbers that we have to keep in mind. :)
@@ -0,0 +1,75 @@ | |||
function BinderBadgeLogo() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In future, I think this can go in icons.
PR that updates myst-theme shared packages to add support for
thebe
See also: executablebooks/myst-book-theme#4
TODO: