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(deps): update dependency rimraf to v5.0.10 #30155

Merged
merged 5 commits into from
Sep 5, 2024

Conversation

gweesin
Copy link
Contributor

@gweesin gweesin commented Aug 31, 2024

Additional details

Steps to test

cd npm/puppeteer

# generate dist
yarn build

# remove dist and generate dist
yarn build

the same as /errors/, /runner/ dir

How has the user experience changed?

No change.

PR Tasks

@CLAassistant
Copy link

CLAassistant commented Aug 31, 2024

CLA assistant check
All committers have signed the CLA.

@cypress-app-bot
Copy link
Collaborator

@MikeMcC399
Copy link
Contributor

@gweesin

There are breaking changes for rimraf from v3 -> v4 -> v5 which need to be accounted for, see https://github.com/isaacs/rimraf/blob/v5.0.10/README.md

The following build command using gulp fails:

yarn workspace @packages/extension build
@packages/extension: [00:00:59] Using gulpfile ~/cypress/packages/extension/gulpfile.ts
@packages/extension: [00:00:59] Starting 'build'...
@packages/extension: [00:00:59] Starting 'clean'...
@packages/extension: [00:00:59] 'clean' errored after 1.15 ms
@packages/extension: [00:00:59] Error: invalid rimraf options
@packages/extension:     at assertRimrafOptions (/root/cypress/node_modules/rimraf/src/opt-arg.ts:23:11)
@packages/extension:     at optArgT (/root/cypress/node_modules/rimraf/src/opt-arg.ts:54:22)
@packages/extension:     at optArg (/root/cypress/node_modules/rimraf/src/opt-arg.ts:82:57)
@packages/extension:     at /root/cypress/node_modules/rimraf/src/index.ts:30:27
@packages/extension:     at clean (/root/cypress/packages/extension/gulpfile.ts:16:9)
@packages/extension:     at bound (node:domain:433:15)
@packages/extension:     at runBound (node:domain:444:12)
@packages/extension:     at asyncRunner (/root/cypress/node_modules/async-done/index.js:55:18)
@packages/extension:     at processTicksAndRejections (node:internal/process/task_queues:77:11)
@packages/extension: [00:00:59] 'build' errored after 8.37 ms

So this means that the following would need to be changed:

const clean = (done) => {
rimraf('dist', done)
}

@jennifer-shehane
Copy link
Member

Running the tests

@gweesin
Copy link
Contributor Author

gweesin commented Sep 4, 2024

Running the tests

I'm sorry my tests running blocked because of my Windows OS(#30157)

@jennifer-shehane
Copy link
Member

@gweesin Can you address the comment left to refactor the code here?

@gweesin
Copy link
Contributor Author

gweesin commented Sep 5, 2024

@gweesin Can you address the comment left to refactor the code here?

Of course. I never use gulp before, so I'm curious if it's possible to receive a Promise instead of callback function. I will try it later

@ryanthemanuel
Copy link
Collaborator

@gweesin Can you address the comment left to refactor the code here?

Of course. I never use gulp before, so I'm curious if it's possible to receive a Promise instead of callback function. I will try it later

@gweesin, Here's the relevant gulp documentation on async functions if you're curious to learn more about it: https://gulpjs.com/docs/en/getting-started/async-completion/#using-asyncawait

@gweesin
Copy link
Contributor Author

gweesin commented Sep 5, 2024

@ryanthemanuel thank you, this is really helpful

@MikeMcC399

This comment was marked as resolved.

@gweesin
Copy link
Contributor Author

gweesin commented Sep 5, 2024

@MikeMcC399 Could I assume that devDependencies are not related to users, so they should be classified as chore(deps) rather than dependency?

@MikeMcC399

This comment was marked as resolved.

@MikeMcC399
Copy link
Contributor

MikeMcC399 commented Sep 5, 2024

@gweesin

My apologies! 🤭 I ran yarn binary-build on your branch and it left rimraf at version 3.0.2, so your commit message is correct.
Please disregard my question about this and I will hide it as "resolved".

Edit: This is caused by

"webpack-dev-server": "^4.15.2",

as [email protected] has a dependency on "rimraf": "^3.0.2"

@jennifer-shehane jennifer-shehane merged commit 7337927 into cypress-io:develop Sep 5, 2024
68 checks passed
@cypress-bot
Copy link
Contributor

cypress-bot bot commented Sep 25, 2024

Released in 13.15.0.

This comment thread has been locked. If you are still experiencing this issue after upgrading to
Cypress v13.15.0, please open a new issue.

@cypress-bot cypress-bot bot locked as resolved and limited conversation to collaborators Sep 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Update to rimraf 4.0.0 or later
7 participants