django-save-the-change icon indicating copy to clipboard operation
django-save-the-change copied to clipboard

Publish to pypi

Open IlyaSemenov opened this issue 7 years ago • 6 comments

The pypi version is almost 3 years old, can you please update it?

IlyaSemenov avatar Mar 13 '17 10:03 IlyaSemenov

There’ve been a whole bunch of changes that I’ve got no feedback on, hence my reluctance to release the version onto PyPI. As far as I know there aren’t any problems with the current version, but my tests and running it in prod are just two datapoints.

Tell you what: If you can run HEAD and don’t see any issues on your end, I’ll push it as a true release. But I don’t want to end up pushing a broken release and get a whole bunch of issues opened up asking why I didn’t test things more thoroughly.

karanlyons avatar Mar 13 '17 22:03 karanlyons

Current version on Pypi triggers an import error with django 1.10:

    from save_the_change.mixins import SaveTheChange
  File "save_the_change/mixins.py", line 11, in <module>
    from django.db.models.related import RelatedObject
ImportError: No module named 'django.db.models.related'

But the latest master version does not, I think it's time to publish the new version

mdamien avatar Jul 12 '17 09:07 mdamien

Yeah, honestly I’m just going to publish the thing at this point. I can almost guarantee that doing so will lead to more bug reports, which I’d much rather get before pushing updates, but eh, die is cast.

karanlyons avatar Jul 13 '17 00:07 karanlyons

+1 and at least you're gonna get good bug reports (against master instead of against an old version)

mdamien avatar Jul 13 '17 13:07 mdamien

+1, my project uses SaveTheChange pretty heavily and all my tests are passing.

oTree-org avatar Aug 08 '17 09:08 oTree-org

+1 - @karanlyons please publish the update to pypi

anirvan-majumdar avatar Oct 07 '19 16:10 anirvan-majumdar