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

ci: fix test failures #234

Merged
merged 13 commits into from
Nov 18, 2020
Merged

ci: fix test failures #234

merged 13 commits into from
Nov 18, 2020

Conversation

pmmmwh
Copy link
Owner

@pmmmwh pmmmwh commented Oct 25, 2020

This PR fixes Chromium for CI - sandboxing is now disabled due to difficulty to have it working reliably (If we had the option to run Docker with more privileges it would work but not at the moment).

This PR fixes multiple stuff for CI:

  • Authenticated Docker Pulls: since 2 Nov, 2020 Docker Hub have imposed rate limiting for image pulls, so auth is added to cope with that (higher rate limit)
  • Using new batch of cimg-puppeteer under the hood now
  • Added tests for Node.js 15 and bumped all LTS versions to latest security patches
  • Bumped webpack@latest and also the corresponding CLI to latest versions

@pmmmwh pmmmwh added the bug Something isn't working label Oct 25, 2020
@pmmmwh pmmmwh self-assigned this Oct 25, 2020
@pmmmwh
Copy link
Owner Author

pmmmwh commented Nov 11, 2020

Scheduled publish of new batch of fixed image at 00:00 UTC - Blocked until that happens and all tests are re-triggered.

@pmmmwh
Copy link
Owner Author

pmmmwh commented Nov 12, 2020

One last remaining issue is on swapping Webpack versions within a child_process, without messing up process.argv for Webpack CLI ... 🤦🏻‍♂️
We might have to rethink how we swap Webpack 4/Webpack 5 versions right now ... sigh.

@pmmmwh
Copy link
Owner Author

pmmmwh commented Nov 12, 2020

Tests now run properly on both Webpack 4 and 5 - and for Webpack 5 they are failing correctly since we haven't caught up to all latest changes from webpack 5.0 to 5.4 ...

@pmmmwh
Copy link
Owner Author

pmmmwh commented Nov 18, 2020

Tests should pass now 🤞🏻

@pmmmwh pmmmwh merged commit eaf46ca into main Nov 18, 2020
@pmmmwh pmmmwh deleted the ci/fix-failures branch November 18, 2020 01:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant