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

chore: update Lerna to version 6 #26521

Closed
wants to merge 10 commits into from
Closed

Conversation

jordanpowell88
Copy link
Contributor

@jordanpowell88 jordanpowell88 commented Apr 17, 2023

Additional details

This updates Lerna from version 3 to version 6

Steps to test

Any package.json script using lerna should still function as expected.

How has the user experience changed?

n/a

PR Tasks

@jordanpowell88 jordanpowell88 changed the title Jordanpowell88/lerna 6 chore: update Lerna to v 6 Apr 17, 2023
@jordanpowell88 jordanpowell88 changed the title chore: update Lerna to v 6 chore: update Lerna to version 6 Apr 17, 2023
lerna.json Outdated
@@ -8,5 +8,6 @@
"system-tests"
],
"useWorkspaces": true,
"useNx": false,
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Once we get to a point where we can flip this switch we will start to see some significant DX and CI improvements! 😄

@cypress
Copy link

cypress bot commented Apr 17, 2023

49 failed and 8 flaky tests on run #45945 ↗︎

49 15985 649 0 Flakiness 8

Details:

chore: revert check-ts changes
Project: cypress Commit: fd2b1aa706
Status: Errored Duration: 39:25 💡
Started: May 1, 2023 8:00 PM Ended: May 1, 2023 8:40 PM
Failed  commands/screenshot.cy.js • 0 failed tests • 5x-driver-firefox

View Output Video

Test Artifacts
Failed  commands/command_option_immutability.cy.js • 0 failed tests • 5x-driver-firefox

View Output Video

Test Artifacts
Failed  e2e/origin/uncaught_errors.cy.ts • 0 failed tests • 5x-driver-firefox

View Output Video

Test Artifacts
Failed  commands/request.cy.js • 0 failed tests • 5x-driver-firefox

View Output Video

Test Artifacts
Failed  e2e/origin/yield.cy.ts • 0 failed tests • 5x-driver-firefox

View Output Video

Test Artifacts

The first 5 failed specs are shown, see all 869 specs in Cypress Cloud.

Flakiness  create-from-component.cy.ts • 2 flaky tests • app-e2e

View Output Video

Test Artifacts
... > runs generated spec Output Screenshots Video
... > runs generated spec Output Screenshots Video
Flakiness  e2e/origin/cookie_login.cy.ts • 1 flaky test • 5x-driver-electron

View Output Video

Test Artifacts
cy.origin - cookie login > Max-Age > past max-age -> not logged in Output Video
Flakiness  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-electron

View Output Video

Test Artifacts
network stubbing > intercepting request > can delay and throttle a StaticResponse Output Video
Flakiness  commands/net_stubbing.cy.ts • 1 flaky test • 5x-driver-chrome

View Output Video

Test Artifacts
network stubbing > intercepting request > can delay and throttle a StaticResponse Output Video
Flakiness  commands/waiting.cy.js • 1 flaky test • 5x-driver-chrome:beta

View Output Video

Test Artifacts
... > errors > throws when waiting for 2nd response to route Output Video

The first 5 flaky specs are shown, see all 6 specs in Cypress Cloud.

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@jordanpowell88 jordanpowell88 marked this pull request as draft April 17, 2023 18:28
@jordanpowell88 jordanpowell88 marked this pull request as ready for review April 17, 2023 19:36
@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/lerna-6 branch 3 times, most recently from 86cc3ea to bd2f3d3 Compare April 18, 2023 12:35
@@ -207,7 +207,7 @@
"through2": "^4.0.2",
"tree-kill": "1.2.2",
"ts-node": "^10.9.1",
"typescript": "^4.7.4",
"typescript": "4.7.4",
Copy link
Member

Choose a reason for hiding this comment

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

why was this needed?

Comment on lines -28 to 29
"inquirer": "7.3.3",
"inquirer": "8.2.4",
"ora": "^5.1.0",
Copy link
Member

Choose a reason for hiding this comment

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

why was this needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This was necessary as the Lerna upgrade has a dependency upon inquirer.

See this release

It was throwing errors when building by leaving it at 7.3.3

Copy link
Member

Choose a reason for hiding this comment

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

What were the inquire breaking changes? Where the scripts in our repo using this dependency tested/check to ensure no other changes were needed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

package.json Outdated
@@ -174,7 +174,7 @@
"inquirer": "3.3.0",
"inquirer-confirm": "2.0.3",
"lazy-ass": "1.6.0",
"lerna": "3.22.1",
"lerna": "6.2.0",
Copy link
Member

Choose a reason for hiding this comment

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

Can we use an open range?

@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/lerna-6 branch from 4bb5352 to 2526f4e Compare April 18, 2023 18:27
@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/lerna-6 branch from c300655 to 413b5bf Compare April 27, 2023 20:34
@emilyrohrbough emilyrohrbough mentioned this pull request Apr 28, 2023
3 tasks
@jordanpowell88 jordanpowell88 force-pushed the jordanpowell88/lerna-6 branch from 211332c to e5bd016 Compare May 1, 2023 19:12
@astone123
Copy link
Contributor

closing in favor of #26660

@astone123 astone123 closed this May 3, 2023
@jordanpowell88 jordanpowell88 deleted the jordanpowell88/lerna-6 branch June 1, 2023 19:47
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.

Upgrade to Lerna 5+ with Nx
4 participants