-
Notifications
You must be signed in to change notification settings - Fork 430
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
Upgrade all packages #551
Upgrade all packages #551
Conversation
We only have devDeps so no need to check them
The tests are failing because GitHub Actions runners use Chrome 98 right now (https://github.com/actions/virtual-environments/blob/main/images/linux/Ubuntu2004-Readme.md#browsers-and-drivers). They also come with ChromeDriver which works with 98. It should be possible to set an environment flag to get |
Could we lock intern to a version that has the prebaked 98 for now? |
f7e67da
to
c637fb2
Compare
Pins Intern chromedriver to the one available in actions Refs theintern/intern#1159 Co-authored-by: Kristján Oddsson <[email protected]>
c637fb2
to
b1cbe12
Compare
CHROMEVER="$(chromedriver --version | cut -d' ' -f2)" | ||
echo "Actions ChromeDriver is $CHROMEVER" | ||
CONTENTS="$(jq '.tunnelOptions.drivers[0].name = "chrome"' < intern.json)" | ||
CONTENTS="$(echo ${CONTENTS} | jq --arg chromever "$CHROMEVER" '.tunnelOptions.drivers[0].version = $chromever')" | ||
echo "${CONTENTS}" > intern.json | ||
cat intern.json | ||
|
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 script uses the actions environment to determine the correct Chrome version, and writes into Intern.json the version to use. I think intern still downloads it's own copy of ChromeDriver, but at least it gets the tests running 😂
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.
Yak riders, saddle up!
This upgrades known dependencies to their latest versions, and makes a few small changes to the
tsconfig
andfetch_request
files to upgrade the failing TypeScript build.