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

update CI config: caching of repo, venv, split test install/run #967

Merged
merged 1 commit into from
Apr 22, 2019

Conversation

protolambda
Copy link
Collaborator

  • repo itself is cached now, for faster checkout
  • venv for test running is cached now
  • venv is installed first now, and tests run separately. Also makes local runs faster if you don't need to re-install dependencies (even if they are cached)
  • make citest does the same as make test, but outputs a test-report for CI
    • Note: the test reporting can be improved, we would want one separate report for each sub-topic. But the tests need to be more structured for that first. This is at least an improvement on status-quo.

@protolambda
Copy link
Collaborator Author

@djrtwo LGTM?

@djrtwo djrtwo merged commit 3ea4620 into dev Apr 22, 2019
@djrtwo djrtwo deleted the ci-tests-config branch April 22, 2019 03:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants