-
Notifications
You must be signed in to change notification settings - Fork 6
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
Towards Continuous Integration #39
Closed
Bengt
wants to merge
155
commits into
DFKI-Interactive-Machine-Learning:main
from
Bengt:main-bengt-continuous-integration
Closed
Towards Continuous Integration #39
Bengt
wants to merge
155
commits into
DFKI-Interactive-Machine-Learning:main
from
Bengt:main-bengt-continuous-integration
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
There does not seem to be a working way of tool-assisted installation of PortAudio under Windows, so I am giving up on that for now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Note that this is a rebase of #37.
This branch adds continuous integration to this project.
To that end, I created a pull mirror of this repository, which gets triggered by GitHub, whenever there is a push to my fork of this repository.
To merge this, someone with the appropriate access rights would need to create a pull mirror for this repository.
The file
.gitlab-ci.yml
defines how to run jobs of pipelines on each push to this repository.As defined by the
tox.ini
, test jobs collect coverage metrics.Currently, the coverage is at only 70 %, because I needed to deactivate some tests to make the test suite work in the cloud. Someone with more knowledge than me about the failing tests would need to make them runnable on AWS.
tox.ini
now also includes a dedicated flake8 environment.This keeps the test environment clean from flake8 requirements.
All environment requirements are pinned in respective files under
requirements.d
.This makes all runs deterministic on the requirements level, which avoids the need for scheduled (e.g. nightly) runs.