Skip to content

Commit

Permalink
Racket 7.7 CS breaks one of the tests. Try Racket 7.8 instead. Update…
Browse files Browse the repository at this point in the history
… the latest-supported version from 7.9 to 8.2.
  • Loading branch information
rocketnia committed Aug 13, 2021
1 parent be5ef41 commit d6f83fe
Showing 1 changed file with 15 additions and 7 deletions.
22 changes: 15 additions & 7 deletions .github/workflows/ci.yml
Expand Up @@ -13,7 +13,11 @@ jobs:
racket-variant: ["BC", "CS"]
# There is no "allow failures" option for GitHub Actions, but we test on
# the latest releases anyway just so we can notice if they break.
racket-version: ["7.7", "7.9", "stable", "current"]
racket-version: ["7.7", "7.8", "8.2", "stable", "current"]
exclude:
# The earliest supported version of Racket CS is 7.8.
- racket-variant: "CS"
racket-version: "7.7"
include:
# This variable is constant across all builds. This is the name of the
# Racket package (`raco pkg install anarki` at the command line) and
Expand Down Expand Up @@ -98,12 +102,10 @@ jobs:
# unknown how well they work.
#
# Versions prior to 6.0.1 don't provide the `--drdr` option, which is used by
# one of the testing commands here, but they otherwise should work as well as
# the versions after.
# one of the testing commands here.
#
# Versions prior to 6.1 don't provide the `--check-pkg-deps` option, which is
# used by one of the testing commands here, but they otherwise should work as
# well as the versions after.
# used by one of the testing commands here.
#
# Versions prior to 6.3 don't define the `racket/random` module, which is needed
# for news.arc and one of the tests.
Expand All @@ -112,9 +114,15 @@ jobs:
# prevents us from having ac.scm statically require `crypto-random-bytes` from
# `racket/random` or `ssl-connect` from `openssl`.
#
# Versions prior to 7.4 didn't have a CS version. (This isn't a problem; it's
# just something that could cause this CI script to fail if we weren't cautious
# about putting versions this old in the version list.)
#
# Versions prior to 7.5 don't provide the `--no-docs` option, which is used by
# the installation commands here, but they otherwise should work as well as the
# versions after.
# the installation commands here.
#
# Versions prior to 7.6.0.7 don't define the `ffi/unsafe/vm` module, which is
# needed to mutate pairs in the Chez-Scheme-based (CS) releases of Racket.
#
# Racket CS versions prior to 7.8 have different `double->string` behavior,
# which breaks one of the tests.

0 comments on commit d6f83fe

Please sign in to comment.