Skip to content

Commit

Permalink
CI: test with GAP 4.13; test with mininimal set of packages loaded (#68)
Browse files Browse the repository at this point in the history
Also fix tests for older GAP versions
  • Loading branch information
fingolfin authored Mar 23, 2024
1 parent c08d93a commit 6dffa0f
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 5 deletions.
10 changes: 7 additions & 3 deletions .github/workflows/CI.yml
Original file line number Diff line number Diff line change
Expand Up @@ -25,19 +25,23 @@ jobs:
matrix:
gap-branch:
- master
- stable-4.13
- stable-4.12
- stable-4.11
- stable-4.10
- stable-4.9

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: gap-actions/setup-gap@v2
with:
GAP_PKGS_TO_BUILD: "io profiling cvec"
GAPBRANCH: ${{ matrix.gap-branch }}
- uses: gap-actions/build-pkg@v1
- uses: gap-actions/run-pkg-tests@v2
- uses: gap-actions/run-pkg-tests@v2
with:
only-needed: true
- uses: gap-actions/process-coverage@v2
- uses: codecov/codecov-action@v3

Expand All @@ -47,13 +51,13 @@ jobs:
runs-on: ubuntu-latest

steps:
- uses: actions/checkout@v3
- uses: actions/checkout@v4
- uses: gap-actions/setup-gap@v2
- uses: gap-actions/build-pkg-docs@v1
with:
use-latex: 'true'
- name: 'Upload documentation'
uses: actions/upload-artifact@v3
uses: actions/upload-artifact@v4
with:
name: manual
path: ./doc/manual.pdf
Expand Down
6 changes: 4 additions & 2 deletions tst/bugfix.tst
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,8 @@ Error, hash function not applicable to key of type integer
# verify equal vectors either have equal hashes, or at least one gives fail
# (here for 8 bit vector rep)
gap> v:=[1..100]*Z(5);;
gap> w:=CopyToVectorRep(v,5);;
gap> w:=[1..100]*Z(5);;
gap> ConvertToVectorRep(w,5);;
gap> v = w;
true
gap> IsPlistRep(v);
Expand All @@ -42,7 +43,8 @@ gap> hf.func(v, hf.data);

# same check for GF(2)
gap> v:=[1..100]*Z(2);;
gap> w:=CopyToVectorRep(v,2);;
gap> w:=[1..100]*Z(2);;
gap> ConvertToVectorRep(w,2);;
gap> v = w;
true
gap> IsPlistRep(v);
Expand Down

0 comments on commit 6dffa0f

Please sign in to comment.