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 node maze-runner version to v7 #1871

Merged
merged 2 commits into from
Nov 17, 2022
Merged

Update node maze-runner version to v7 #1871

merged 2 commits into from
Nov 17, 2022

Conversation

Cawllec
Copy link
Contributor

@Cawllec Cawllec commented Nov 17, 2022

No description provided.

@github-actions
Copy link

github-actions bot commented Nov 17, 2022

@bugsnag/browser bundle size diff

Minified Minfied + Gzipped
Before 43.19 kB 13.21 kB
After 43.19 kB 13.21 kB
± No change No change

code coverage diff

Coverage values did not change👌.

Total:

Lines Branches Functions Statements
0%(+0%) 0%(+0%) 0%(+0%) 0%(+0%)

Generated by 🚫 dangerJS against 11081b1

@@ -21,7 +21,7 @@ RUN npm pack --verbose packages/plugin-koa/
RUN npm pack --verbose packages/plugin-restify/

# The maze-runner node tests
FROM 855461928731.dkr.ecr.us-west-1.amazonaws.com/maze-runner-releases:latest-v6-cli as node-maze-runner
FROM 855461928731.dkr.ecr.us-west-1.amazonaws.com/maze-runner-releases:latest-v7-cli as node-maze-runner
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I don't know if anyone runs the Node tests locally, but I think we should also update the Gemfilein test/node.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've updated that, as well as a typo which was triggering a non-fatal step missing

Copy link
Contributor

@twometresteve twometresteve left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM pending CI

@Cawllec Cawllec merged commit 8d607ca into next Nov 17, 2022
@Cawllec Cawllec deleted the testing/node-maze-v7 branch November 17, 2022 14:33
djskinner added a commit that referenced this pull request Nov 22, 2022
* next:
  Update node maze-runner version to v7 (#1871)
djskinner added a commit that referenced this pull request Nov 29, 2022
* next:
  Update node maze-runner version to v7 (#1871)
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.

2 participants