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

Release 1.0.0 #69

Merged
merged 16 commits into from Aug 23, 2020
Merged

Release 1.0.0 #69

merged 16 commits into from Aug 23, 2020

Conversation

dsaxton
Copy link
Collaborator

@dsaxton dsaxton commented Aug 10, 2020

@HDembinski Couple edits to setup.py before releasing the new version (I can also make you co-owner on PyPI). Let me know if there are any other loose ends you can think of that need to be taken care of before releasing (it's been two years since the last time I did this and need to refresh my memory).

Copy link
Member

@HDembinski HDembinski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should add a changelog, since we made major changes. I can help writing that, if you like.

Otherwise it looks good to me, we already updated the version. Thank you for adding me to the authors. :)

For releases, I have usually a release guide in my repositories, since I also tend to forget the procedure. It may be a good idea to write one and add it to this PR?

@HDembinski
Copy link
Member

@dsaxton If you could add me also to PyPI that would be great, thanks!

@HDembinski
Copy link
Member

For reference, here is the release guide for iminuit: https://github.com/scikit-hep/iminuit/blob/develop/doc/README.md

@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 11, 2020

@HDembinski Agree about the changelog (I'll add that and start working on it soon). What's your username on PyPI? I'll add you as a collaborator. Nice that we can release through GitHub, that simplifies the process a lot.

@dsaxton dsaxton changed the title Update setup.py for release Release 1.0.0 Aug 11, 2020
@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 11, 2020

Started changelog.rst with a few of the changes if you wanted to add some notes about the interface overhaul (might want to have a section that specifically calls out that these are breaking changes).

@HDembinski
Copy link
Member

HDembinski commented Aug 12, 2020

Sorry, I should have said it is hdembins on PyPI.

@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 12, 2020

Sorry, I should have said it is hdembins on PyPI.

Should be added as maintainer now

@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 13, 2020

@HDembinski feel free to make / propose edits to the changelog, I'm trying to document most everything from the closed issues

doc/changelog.rst Outdated Show resolved Hide resolved
doc/changelog.rst Outdated Show resolved Hide resolved
doc/changelog.rst Outdated Show resolved Hide resolved
@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 20, 2020

@HDembinski Are you good with the minor suggested edits? Would be nice to get this out if possible (I'll likely need some help with that).

@HDembinski
Copy link
Member

Yes, all good. Sorry for being not very responsive in the last weeks, work and private life are eating up my spare time currently. Please don't wait for my feedback for the release. I think it got great!

@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 22, 2020

@HDembinski Following the steps here https://packaging.python.org/guides/publishing-package-distribution-releases-using-github-actions-ci-cd-workflows/ and it seems we should probably have a pyproject.toml file. I think you mentioned this in a previous issue in regard to packaging "best practices"; are you familiar with how to set this up by chance?

@HDembinski
Copy link
Member

Not really, but I can research it.

@dsaxton
Copy link
Collaborator Author

dsaxton commented Aug 23, 2020

@HDembinski Uploaded to PyPI and I think we're good to go:

Python 3.8.5 (default, Aug  5 2020, 03:39:04)
[Clang 10.0.0 ] :: Anaconda, Inc. on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> import resample
>>> resample.__version__
'1.0.0'

Also, there are a bunch of junk releases for this package still in PyPI. Would you be opposed to us simply deleting those (I was actually experimenting with it at the time)?

@dsaxton dsaxton merged commit 7f7f248 into master Aug 23, 2020
@dsaxton dsaxton deleted the update-setup branch August 23, 2020 00:32
@HDembinski
Copy link
Member

@dsaxton Deleting (yanking) old stuff is fine with me, but most people will never notice it is there. It is up to you :)

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

Successfully merging this pull request may close these issues.

None yet

2 participants