-
Notifications
You must be signed in to change notification settings - Fork 3
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
Initial CI #3
Initial CI #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A couple minor changes. Should be quick to get this in and iterate
name: CI | ||
|
||
on: | ||
push: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
add pull_request
as a trigger as well
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we have specific PR events we want to run on or just all PR events?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The default is opened
, reopened
, and synchronize
which should be enough for this and the changes have been pushed.
.github/workflows/ci.yaml
Outdated
python -m pip install --upgrade pip setuptools wheel | ||
python -m pip install --no-cache-dir . | ||
python -m pip install --no-cache-dir '.[test]' | ||
cd tests/ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Probably move this down to be part of the Run tests
step. Also, this will break the current pytest
command (since it would now be running tests in the tests/tests/
directory).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Let's merge it and hope it runs....
This PR adds an initial GitHub action for use in testing this project. It will later be adapted to handle
mypy
and linting down the line.