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 show_banner to ignored_traits #29

Merged
merged 2 commits into from
Apr 24, 2023

Conversation

echarles
Copy link
Member

@echarles echarles commented Apr 17, 2023

Contributes to jupyter/nbclassic#242

Once this is merged and released, jupyter/nbclassic#243 needs to be upated.

@echarles
Copy link
Member Author

@Zsailer @jtpio any chance to get this reviewed? We will need a release after, any volunteer for that? (I don't know if I have the rights for a release).

@jtpio
Copy link
Member

jtpio commented Apr 24, 2023

Looks like the CI failure might not be relevant, but would still be good to address before a new release?

@echarles
Copy link
Member Author

Thx @jtpio CI is green now.

@Zsailer
Copy link
Member

Zsailer commented Apr 24, 2023

Looks good. Thanks @echarles

@Zsailer Zsailer merged commit bb84e64 into jupyter:main Apr 24, 2023
@echarles
Copy link
Member Author

Thx @Zsailer Is it possible to get a release with this PR?

@Zsailer
Copy link
Member

Zsailer commented Apr 24, 2023

Yeah, happy to cut the release.

You should have permissions too—it looks like all notebook team members are listed as admin on this repo.

@echarles
Copy link
Member Author

echarles commented Apr 24, 2023

I am not listed on pypi for notebook_shim, I let you cut the release this time. Thx.

@Zsailer
Copy link
Member

Zsailer commented Apr 24, 2023

Ah, I see. Just added you as a maintainer on the PyPi package.

Released: https://github.com/jupyter/notebook_shim/releases/tag/v0.2.3

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants