search

lambdamusic / Ontospy

Python library and command-line interface for inspecting and visualizing RDF models aka ontologies.
http://lambdamusic.github.io/Ontospy/
MIT License
219 stars 52 forks source link

Add Continuous Integration to assist with code review #100

Closed ajnelson-nist closed 2 years ago

ajnelson-nist commented 2 years ago

This patch series adds CI, making use of Github Actions. Pull Requests against the master branch will trigger CI and give an indicator of whether there is a unit tests issue, before any manual review is needed. (Asterisk: Github has started requiring first-time contributors get their CI workflow approved to run, so there is one repo.-admin click to do per user before the CI runs.)

Overall, this patch series is expected to reduce the amount of round-trip communications between the project maintainer and pull request submitters, as a submitter can handle automatically-discovered build issues before requesting the maintainer's review.

Disclaimer: Participation by NIST in the creation of the documentation of mentioned software is not intended to imply a recommendation or endorsement by the National Institute of Standards and Technology, nor is it intended to imply that any specific software is necessarily the best available for the purpose.

Signed-off-by: Alex Nelson alexander.nelson@nist.gov

ajnelson-nist commented 2 years ago

For reference, I've run the Github Actions for this patch series here: https://github.com/ajnelson-nist/Ontospy/pull/1

ajnelson-nist commented 2 years ago

Another benefit of adding CI is fairly quick visibility into which versions of Python can be supported. This reference PR shows this repository functions in Pythons 3.8 and 3.9.

lambdamusic commented 2 years ago

Honestly, I have never used much Github CI actions. But it looks very useful, especially for detecting failing PRs :-) I'll give it a go, thanks for suggesting.