search

jupyter / jupyter_events

Configurable event system for Jupyter applications and extensions.
https://jupyter-events.readthedocs.io/en/latest/
BSD 3-Clause "New" or "Revised" License
12 stars 21 forks source link

use safe loaders and dumpers in yaml lib #28

Closed dlqqq closed 2 years ago

dlqqq commented 2 years ago

See #26.

safe_load(stream) is really just invoking load(stream, Loader=SafeLoader), and similarly for safe_dump(stream) (see source). I've preserved the existing behavior of trying to use the (much faster) C implementation before the Python implementation.

dlqqq commented 2 years ago

@Zsailer I'll need to bump the jupyter_events version in Jupyter Server after this, right?

Zsailer commented 2 years ago

Thanks, @dlqqq. Great stuff!

Yes, we'll need to make a new release of jupyter_events and bump in jupyter_server.

I'll cut a release on Monday morning (I avoid releasing on Fridays to avoid possible hotfix scenarios on weekends 😅 ).