-
Notifications
You must be signed in to change notification settings - Fork 7
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
Add GitHub action for CI tests #31
Add GitHub action for CI tests #31
Conversation
language: c | ||
env: | ||
global: | ||
- GAP_PKGS_TO_BUILD="io profiling orb" |
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.
This PR does not seem to have transferred the above (orb
is missing)
- env: GAPBRANCH=master | ||
- env: GAPBRANCH=stable-4.11 | ||
- env: GAPBRANCH=stable-4.10 | ||
- env: GAPBRANCH=master ABI=32 |
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.
this 32bit test is missing, too
README.md
Outdated
@@ -1,4 +1,4 @@ | |||
[![Build Status](https://travis-ci.com/gap-packages/cvec.svg?branch=master)](https://travis-ci.com/gap-packages/cvec) | |||
![CI](https://github.com/gap-packages/cvec/workflows/CI/badge.svg) |
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.
This is missing a link to the CI results (as it is present in e.g. the GAP README.md)
0767f4f
to
d9d279e
Compare
@fingolfin |
d9d279e
to
73a1bad
Compare
@fingolfin Now tests passed. |
This PR configures GitHub Actions to run the package tests. It was created by a script. You can view the results of the added GitHub action at:
https://github.com/ssiccha/cvec/actions
This PR contains four commits which do:
.github/workflows/CI.yml
which configures GitHub Actions to run the package tests,.travis.yml
file if it exists in the package,Since this PR is generated automatically, before merging it:
GAP_PKGS_TO_BUILD
andGAP_PKGS_TO_CLONE
in the.github/workflows/CI.yml
file as described further below..travis.yml
but are missing in the.github/workflows/CI.yml
.The added GitHub action
CI
contains a jobtest
, which runs your package's tests using the same scripts as previously did Travis, namely those fromhttps://github.com/gap-system/pkg-ci-scripts
By default it tests your package with the following gap branches:
The action
CI
also contains a jobmanual
, which compiles your package's documentation with latex and uploads the resulting pdf as an artifact, which means that it can be downloaded once all jobs of the action completed.If you need to compile packages to load your package, set the input
GAP_PKGS_TO_BUILD
for the actiongap-actions/setup-gap-for-packages@v1
to a space-separated list of packages as follows:If you need to clone the development version of packages set the input
GAP_PKGS_TO_CLONE
for the actiongap-actions/setup-gap-for-packages@v1
to a space-separated list of packages as follows:Notice that you may also have to set inputs for the action
gap-actions/setup-gap-for-packages@v1
in the jobmanual
.If you want to adjust which tests are run have a look at the documentation of the job matrix, especially also at how to use the
include:
keywordYou can add changes to this PR by adding my fork as a remote, checking it out and then committing and pushing as follows:The custom gap actions which are used by the added action can be found at:
https://github.com/gap-actions