search

ORCID / python-orcid

Python wrapper around ORCID API
BSD 3-Clause "New" or "Revised" License
91 stars 26 forks source link

Restructuring package #13

Closed kaplun closed 6 years ago

kaplun commented 8 years ago

At Invenio big effort has been put into building a well structure package: https://github.com/inveniosoftware/cookiecutter-invenio-module

The best practices that went into that template can be ported to python-orcid. E.g. Sphynx documentation, e.g. separation of tests (so that coverage covers only the actual code), etc.

MSusik commented 8 years ago

Can be done, but not crucial

kaplun commented 8 years ago

I plan to send a PR soon :)

kaplun commented 8 years ago

As part of this, I would like to propose adding auto-pushing of packages from Travis to Pypi, based on tagged version.

(e.g. see: https://github.com/inspirehep/beard-server/blob/master/.travis.yml#L20)

Since when passing this project under the ORCID organization on Github we agreed that CERN (in the form of INSPIRE project) would have still stayed co-maintainer, could you add to the authorized accounts on Pypi also the 'inspirehep' account? In this way I can set up the auto pushing functionality.

kaplun commented 8 years ago

Can be done, but not crucial

Maybe not directly crucial, but at INSPIRE we are more and more relying on this package as part of the Labs project, definitively putting it to good use, so are interested into having it consistent across the ecosystem of other packages.

MSusik commented 6 years ago

Closing issue due to no activity.