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 binder icon #6

Merged

Conversation

agoose77
Copy link
Contributor

@agoose77 agoose77 commented Feb 5, 2025

This PR adds a black-and-white icon for the Binder project.
It's not clear to whom the Binder logo belongs — e.g. can it be used for BinderHub in general, but I think my feeling is yes.

I intend to use this in jupyter-book/myst-theme#528. I have not contributed an icon here before, so I feel like I must have missed something.

@@ -1,111 +1,113 @@
// node_modules/nanoid/index.js
import { randomFillSync } from "crypto";
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A load of LSP-derived linting. I can remove this if you don't like the changes.

@agoose77
Copy link
Contributor Author

agoose77 commented Feb 6, 2025

@stevejpurves would you be able to review this PR? :)

@rowanc1 rowanc1 merged commit 4295af6 into continuous-foundation:main Feb 6, 2025
@rowanc1
Copy link
Member

rowanc1 commented Feb 6, 2025

Released as 0.0.12

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