jeetsukumaran / DendroPy

A Python library for phylogenetic scripting, simulation, data processing and manipulation.
https://pypi.org/project/DendroPy/.
BSD 3-Clause "New" or "Revised" License
205 stars 63 forks source link

Setup CI #157

Closed mmore500 closed 1 year ago

mmore500 commented 1 year ago

Replacement PR for #156

TODO:

mmore500 commented 1 year ago

In order for deployment-from-CI to work, we will need to get a PyPi API token and store it as a GitHub secret associated with this repository. Here are instructions for how to do that:

mmore500 commented 1 year ago

Looks like this is ready to review! The big changes were

  1. configuring .github/workflows/ci.yaml to set up CI (see example CI run here: https://github.com/jeetsukumaran/DendroPy/actions/runs/4590910247)
  2. setting up bumpver integration to streamline the version bump/release process (i.e., bumpver.toml)
  3. reverting __version__ to the last version published on PyPi
    • it looks like a calendaring scheme was being used for the minor version?
    • it would be easy to reconfigure bumpver to use this scheme if we want to stick with it
  4. adding a developer guide to docs with instructions on how lint, test, release, etc.
  5. some diffuse cleanup work to get the linter and unit tests passing
    • ideally, these shouldn't be changing the behavior of anything that already worked
    • it would definitely be worth a second eye on these as I'm still learning my way around the project

I figure after this gets integrated we might bump and release a new minor version (4.6.0) on the main branch. All that we'll need in order to do that is to configure the PYPI_API_TOKEN secret for the repository (see notes above).

codecov-commenter commented 1 year ago

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@ad018c8). Click here to learn what that means. The diff coverage is n/a.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@           Coverage Diff           @@
##             main     #157   +/-   ##
=======================================
  Coverage        ?   60.75%           
=======================================
  Files           ?      105           
  Lines           ?    18646           
  Branches        ?        0           
=======================================
  Hits            ?    11329           
  Misses          ?     7317           
  Partials        ?        0           
Flag Coverage Δ
unittests 60.75% <0.00%> (?)

Flags with carried forward coverage won't be shown. Click here to find out more.

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.