search

kynan / nbstripout

strip output from Jupyter and IPython notebooks
Other
1.19k stars 95 forks source link

Tags but not releases; rev: master is not supported #145

Closed dmyersturnbull closed 3 years ago

dmyersturnbull commented 3 years ago

This is a fantastic package! Thanks for continuing to update it.

I was originally using v0.3.1 from the releases, but 0.3.9 is available as a tag. I had just assumed that 0.3.1 was the most recent! Is there a reason why there are no releases corresponding to recent tags?

Related, the example in the readme uses rev: master, which causes pre-commit to emit this warning:

Warning:  The 'rev' field of repo 'https://github.com/kynan/nbstripout' appears to be a mutable reference (moving tag / branch).  Mutable references are never updated after first install and are not supported.  See https://pre-commit.com/#using-the-latest-version-for-a-repository for more details.
kynan commented 3 years ago

I was originally using v0.3.1 from the releases, but 0.3.9 is available as a tag. I had just assumed that 0.3.1 was the most recent! Is there a reason why there are no releases corresponding to recent tags?

None other than oversight on my part. I've turned all tags into "proper" releases now, thanks for pointing this out.

Is there a reason you're not installing via pip or conda?

Related, the example in the readme uses rev: master, which causes pre-commit to emit this warning:

Warning:  The 'rev' field of repo 'https://github.com/kynan/nbstripout' appears to be a mutable reference (moving tag / branch).  Mutable references are never updated after first install and are not supported.  See https://pre-commit.com/#using-the-latest-version-for-a-repository for more details.

I think I was just being lazy and didn't want to forget updating to the latest tag, but I can just configure bumpversion to do that for me. I'll push a fix, thanks for flagging.