danielfrg / s3contents

Jupyter Notebooks in S3 - Jupyter Contents Manager implementation
Apache License 2.0
248 stars 88 forks source link

Save implementation does not call run_pre_save_hook #65

Closed myu07 closed 4 years ago

myu07 commented 5 years ago

Is there a reason why your "save" implementation does not call run_pre_save_hook? I have a usecase where I can use this. Thanks.

danielfrg commented 5 years ago

No real reason and I can't think why it should not work. Could you make a PR that adds it?

ericdill commented 4 years ago

Closed by #97