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

Enable type checking in Continuous Integration #250

Merged

Conversation

ajnelson-nist
Copy link
Contributor

This Pull Request adds type checking to a continuous integration (CI) process outside of the two CI systems currently encoded in this repository. The CI workflow set up uses GitHub Actions, but in a matter naïve to other systems that have some scripting written in this repository. E.g., I'm not familiar with tox.

Feedback or revisions are welcome on this PR. My objective with the PR is to type-check code on the way in. I'm aware that there are other tests, but I'm not sure how they're all meant to be called, so the PR starts small: Check types on the oldest and newest Python releases designated in pyproject.toml.

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.

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 <[email protected]>
@ajnelson-nist
Copy link
Contributor Author

Ah, and apparently, some setting may need to be turned on in order to run GitHub Actions on this repository (possibly Settings / Actions / General / Allow [x] actions and reusable workflows). IIRC, closing the PR and re-opening it will trigger an Action run.

@ajnelson-nist ajnelson-nist marked this pull request as ready for review September 25, 2024 15:29
@ashleysommer
Copy link
Collaborator

@ajnelson-nist

may need to be turned on in order to run GitHub Actions on this repository (possibly Settings / Actions / General / Allow [x] actions and reusable workflows

The option "Allow all actions and reusable workflows" is already enabled.

@ajnelson-nist
Copy link
Contributor Author

Is there a checkbox, or "Run first Action" interface, at the bottom of this PR in the view of any project Administrators?

If not: I have a hunch that the CI not firing is an anti-abuse mechanism. I'm not a project maintainer, so I probably can't initiate the Actions infrastructure. I think if a project maintainer adds an Actions YAML workflow file first, CI would start firing on this.

I'd thought project Contributors could get by this, but it seems no, & in the bigger picture that's fine by me.

Maybe just the first 36 lines of the YAML in this PR should be merged first? I'll file another PR.

ajnelson-nist added a commit to ajnelson-nist/pySHACL that referenced this pull request Sep 30, 2024
This patch is an attempt to enable GitHub Actions.  It is a reduced
implementation of the patch in PR 250, and intentionally doesn't run
tests.

References:
* RDFLib#250

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 <[email protected]>
@ashleysommer
Copy link
Collaborator

After merging #252, the tests on here are now running!

@ashleysommer
Copy link
Collaborator

Passed type checking!

@ashleysommer ashleysommer merged commit ab4bf12 into RDFLib:master Sep 30, 2024
2 checks passed
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.

2 participants