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

Update codeceptjs #152

Merged
merged 2 commits into from
Jun 23, 2023
Merged

Update codeceptjs #152

merged 2 commits into from
Jun 23, 2023

Conversation

frederikprijck
Copy link
Contributor

We are using an incorrect docker image that hasn't been updated for 3 years: hub.docker.com/r/codeception/codeceptjs

Instead, we should be using hub.docker.com/r/codeceptjs/codeceptjs instead.

We are using an incorrect docker image that hasn't been updated for 3 years: hub.docker.com/r/codeception/codeceptjs

Instead, we should be using hub.docker.com/r/codeceptjs/codeceptjs instead.
@frederikprijck
Copy link
Contributor Author

Marking this as draft as the latest codeceptjs/codeceptjs uses a new major version of codeceptjs which requires changes. We have applied these changes on the chore/update-deps branch for our tests.

Once all samples have been updated and CI succeeds, we will merge the quickstart test changes to master and update this PR.

@frederikprijck frederikprijck marked this pull request as ready for review June 23, 2023 07:09
@frederikprijck frederikprijck requested a review from a team as a code owner June 23, 2023 07:09
@frederikprijck frederikprijck enabled auto-merge (squash) June 23, 2023 07:30
@frederikprijck frederikprijck merged commit 1febae5 into master Jun 23, 2023
5 checks passed
@frederikprijck frederikprijck deleted the frederikprijck-patch-2 branch June 23, 2023 08:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants