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

Versioning and pypi upload ? #33

Closed
octomike opened this issue Oct 16, 2019 · 21 comments
Closed

Versioning and pypi upload ? #33

octomike opened this issue Oct 16, 2019 · 21 comments

Comments

@octomike
Copy link

For new users it's a bit confusing to see what version to use. Pypi only has 1.0.0 (which is old and still uses python2), Github releases only list 1.0.0 and 1.1.0 and setup.py contains version string 2.0.

A new 2.0 (pre) release(?), a tag and an upload to pypi would be super helpful for us :)

@vsoch
Copy link
Contributor

vsoch commented Oct 16, 2019

Totally agree, +1 on this one. 2.0 also is not using semantic versioning, which is important to do.

@ofgulban
Copy link
Collaborator

ofgulban commented Oct 30, 2019

+1. It would also be good to know who are the active maintainers, @chrisgorgo @effigies ? I am time to time getting emails about errors etc. although I am not a maintainer. I would be willing to help tidying things up if you add me as a collaborator.

@vsoch
Copy link
Contributor

vsoch commented Oct 30, 2019

I left the lab when I graduated in 2016 so I wouldn't be the right fit, and I know @chrisgorgo is now at Google, so he likely can't take it on. Pinging @rwblair and @oesteban who will likely know!

@effigies
Copy link
Contributor

Looks like @poldrack is the maintainer on PyPI, so he'll need to either upload a 1.1.0 package or add maintainers.

As someone with essentially no connection to this project, I would be 👍 for adding @ofgulban as a maintainer on the strength of #15.

@poldrack
Copy link
Contributor

poldrack commented Oct 30, 2019 via email

@vsoch
Copy link
Contributor

vsoch commented Oct 30, 2019

Also a note for everyone - pypi this year has added two factor auth (and are also working on other security measures) so anyone that has an account that can manage packages, in that someone getting access and fussing around would have dire consequences, please strongly consider using it!

@poldrack
Copy link
Contributor

poldrack commented Oct 30, 2019 via email

@ofgulban
Copy link
Collaborator

if @ofgulban is happy to do that then I'm happy to add him

Yes I am happy to help @poldrack .

@poldrack
Copy link
Contributor

poldrack commented Oct 30, 2019 via email

@effigies
Copy link
Contributor

@ofgulban Gave you write access to the repository. Let me know if you need that bumped to admin.

@ofgulban
Copy link
Collaborator

my pypi username is ofgulban (https://pypi.org/user/ofgulban/)

@ofgulban
Copy link
Collaborator

@ofgulban Gave you write access to the repository. Let me know if you need that bumped to admin.

Thanks @effigies . I think it is ok for now. If need it I will let you know.

@ofgulban
Copy link
Collaborator

@effigies could you bump me to admin? I need it to setup zenodo.

Also, I think it would be safe to limit write access to master branch after releasing 2.0.0 without further change in the code and develop in devel branch in the future. I can also handle this.

@effigies
Copy link
Contributor

@ofgulban Done.

@ofgulban
Copy link
Collaborator

Thanks @effigies zenodo badge is ready.

@poldrack it seems that I do not have the right permissions to upload to pypi. My pypi username is ofgulban (https://pypi.org/user/ofgulban/), in case if you have missed the message above.

@poldrack
Copy link
Contributor

poldrack commented Oct 31, 2019 via email

@ofgulban
Copy link
Collaborator

@poldrack thanks. Could you also add me as a maintainer in https://test.pypi.org/ . It would be cleaner to check if everything looks alright there.

@effigies
Copy link
Contributor

effigies commented Oct 31, 2019

@ofgulban The first uploader becomes maintainer on regular PyPI, so that is likely to be true on the test server. Since pydeface isn't on on there, you should be able to do it yourself.

@ofgulban
Copy link
Collaborator

Oh I see.

@poldrack
Copy link
Contributor

poldrack commented Oct 31, 2019 via email

@ofgulban
Copy link
Collaborator

  1. I did some cosmetic changes in readme and setup.py. Did not touch to any code for now.
  2. I have fixed the versioning. It is 2.0.0 now.
  3. Setup zenodo integration. See https://zenodo.org/record/3524401.
  4. Uploaded to pypi (https://pypi.org/project/pydeface/>). Seems to work fine

Thanks @octomike & @vsoch for highlighting these issues. Thanks @poldrack and @effigies for your quick responses.

I will have a look at the remaining issues and pull requests wheneverly, but probably after ISMRM deadline next week unless if there is sth broken.

@ofgulban ofgulban closed this as completed Nov 1, 2019
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

No branches or pull requests

5 participants