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 v4 #7907

Merged
merged 7 commits into from
Mar 25, 2023
Merged

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Mar 24, 2023

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
rimraf 3.0.2 -> 4.4.1 age adoption passing confidence

Release Notes

isaacs/rimraf

v4.4.1

Compare Source

v4.4.0

Compare Source

v4.3.1

Compare Source

v4.3.0

Compare Source

v4.2.0

Compare Source

v4.1.4

Compare Source

v4.1.3

Compare Source

v4.1.2

Compare Source

v4.1.1

Compare Source

v4.1.0

Compare Source

v4.0.7

Compare Source

v4.0.6

Compare Source

v4.0.5

Compare Source

v4.0.4

Compare Source

v4.0.3

Compare Source

v4.0.2

Compare Source

v4.0.1

Compare Source

v4.0.0

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.


  • If you want to rebase/retry this PR, check this box

This PR has been generated by Mend Renovate. View repository job log here.

@renovate renovate bot added the release:chore This PR is a chore (means nothing for users) label Mar 24, 2023
@renovate renovate bot assigned jtoar Mar 24, 2023
@jtoar
Copy link
Contributor

jtoar commented Mar 24, 2023

We need to check all package build scripts; many of them use rimraf to get rid of test files.

@renovate renovate bot force-pushed the renovate/rimraf-4.x branch 11 times, most recently from 951ad38 to 7f4e96b Compare March 24, 2023 23:34
@renovate renovate bot force-pushed the renovate/rimraf-4.x branch from 7f4e96b to 7e1640b Compare March 25, 2023 00:06
@replay-io
Copy link

replay-io bot commented Mar 25, 2023

16 replays were recorded for 1f9a778.

image 0 Failed
image 16 Passed
    requireAuth graphql checks
          ```
          locator.waitFor: Target closed
          =========================== logs ===========================
          waiting for locator('.rw-form-error-title').locator('text=You don\'t have permission to do that') to be visible
          ============================================================
          ```
        </ol>
      </details>
      <li><a href=https://app.replay.io/recording/85db418b-9f58-4d68-8b79-a3fd53e95bfa>useAuth hook, auth redirects checks</a></li>
      <li><a href=https://app.replay.io/recording/ba492e27-5604-426a-91cb-1c7247bce8dd>Check that a specific blog post is prerendered</a></li>
      <li><a href=https://app.replay.io/recording/6a03d283-3555-43f5-a2d1-b60456a8e879>Check that about is prerendered</a></li>
      <li><a href=https://app.replay.io/recording/0531d986-40c2-47dd-b780-0217ba9c6a07>Check that homepage is prerendered</a></li>
      <li><a href=https://app.replay.io/recording/9a7c6a3e-c163-4ce0-b4b7-858acb1779cf>Check that meta-tags are rendering the correct dynamic data</a></li>
      <li><a href=https://app.replay.io/recording/f1219194-9ee1-493b-8dda-0d5e7ca18645>Check that you can navigate from home page to specific blog post</a></li>
      <li><a href=https://app.replay.io/recording/6551ebdb-66f9-45ba-951e-2c6b134b10f3>Waterfall prerendering (nested cells)</a></li>
      <li><a href=https://app.replay.io/recording/2782ae6b-c7a8-48a4-8a27-f6679b7febe2>RBAC: Admin user should be able to delete contacts</a></li>
      <li><a href=https://app.replay.io/recording/42fb578f-6287-40c2-8fd5-3dc0341d5be0>RBAC: Should not be able to delete contact as non-admin user</a></li>
      <li><a href=https://app.replay.io/recording/5dba1c30-615f-4e8b-8aa0-61e00fde3ec3>Smoke test with dev server</a></li>
      <li><a href=https://app.replay.io/recording/334716b5-1cfa-40b0-8807-ab322a6d8057>Smoke test with rw serve</a></li>
      <li><a href=https://app.replay.io/recording/468e1de6-1e60-4805-b7dc-0e4492858210>Loads Cell mocks when Cell is nested in another story</a></li>
      <li><a href=https://app.replay.io/recording/30f8c289-14c2-4401-9ad2-703520db007c>Loads Cell Stories</a></li>
      <li><a href=https://app.replay.io/recording/8e3b7eb2-b2a1-4342-a3a0-4ad511470543>Loads MDX Stories</a></li>
      <li><a href=https://app.replay.io/recording/749706ba-e826-445e-9b94-3faebbf01776>Mocks current user, and updates UI while dev server is running</a></li>
      

View test run on Replay ↗︎

@renovate renovate bot force-pushed the renovate/rimraf-4.x branch from 7e1640b to 68c650a Compare March 25, 2023 00:44
@thedavidprice
Copy link
Contributor

Testing locally, including test-project and project yarn rw build.

Subjectively await rimraf() might be a bit faster than rimraf.sync()

@thedavidprice thedavidprice requested a review from jtoar March 25, 2023 01:52
@@ -7,7 +7,7 @@
],
"scripts": {
"build": "lerna run build",
"build:clean": "yarn clean:prisma && rimraf \"packages/**/dist\"",
"build:clean": "yarn clean:prisma && rimraf \"packages/**/dist\" --glob",
Copy link
Contributor

Choose a reason for hiding this comment

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

new config required. Note: I tested locally to confirm working, especially in case of test directories

@renovate
Copy link
Contributor Author

renovate bot commented Mar 25, 2023

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

Warning: custom changes will be lost.

@@ -89,7 +89,7 @@ export const handler = async ({
// Vite handles this internally
title: 'Cleaning Web...',
task: () => {
rimraf.sync(rwjsPaths.web.dist)
rimraf(rwjsPaths.web.dist)
Copy link
Contributor

Choose a reason for hiding this comment

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

rimraf() now returns promise. rimraf.sync() is slower

@@ -117,7 +117,7 @@ export async function copyFrameworkFilesToProject(
files.length,
'files'
)
rimraf.sync(packageDstPath)
await rimraf(packageDstPath)
Copy link
Contributor

Choose a reason for hiding this comment

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

using new return Promise behavior

@@ -9658,7 +9656,7 @@ __metadata:
languageName: node
linkType: hard

"@types/rimraf@npm:3.0.2, @types/rimraf@npm:^3.0.2":
"@types/rimraf@npm:^3.0.2":
Copy link
Contributor

Choose a reason for hiding this comment

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

hmm, this shouldn't be here anymore as types are included in v4 Checking

Copy link
Contributor

Choose a reason for hiding this comment

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

It's ok. Required by the google-gax dep

@thedavidprice thedavidprice enabled auto-merge (squash) March 25, 2023 03:20
@thedavidprice thedavidprice merged commit e1098c4 into main Mar 25, 2023
@thedavidprice thedavidprice deleted the renovate/rimraf-4.x branch March 25, 2023 03:52
@redwoodjs-bot redwoodjs-bot bot added this to the next-release milestone Mar 25, 2023
@jtoar jtoar modified the milestones: next-release, v5.0.0 Mar 27, 2023
jtoar added a commit that referenced this pull request Mar 27, 2023
* chore(deps): update dependency rimraf to v4

* chore(deps): update dependency rimraf to v4

* codemod breaking changes

* Update packages/cli/src/commands/buildHandler.js

---------

Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: David Price <[email protected]>
Co-authored-by: Dominic Saadi <[email protected]>
dac09 added a commit to dac09/redwood that referenced this pull request Mar 28, 2023
* 'main' of github.com:redwoodjs/redwood: (162 commits)
  fix(deps): update dependency @escape.tech/graphql-armor to v1.8.0 (redwoodjs#7922)
  chore(deps): update dependency @playwright/test to v1.32.1 (redwoodjs#7924)
  chore(deps): update dependency @testing-library/dom to v9.2.0 (redwoodjs#7921)
  chore(deps): update dependency rimraf to v4 (redwoodjs#7907)
  fix(deps): update dependency react-hook-form to v7.43.8 (redwoodjs#7920)
  fix(deps): update dependency @fastify/http-proxy to v9 (redwoodjs#7889)
  fix(deps): update dependency @types/node to v16.18.20 (redwoodjs#7919)
  chore(deps): update dependency @replayio/playwright to v0.3.27 (redwoodjs#7918)
  fix(Clerk): call reauthenticate before navigating (redwoodjs#7917)
  chore(deps): update dependency esbuild to v0.17.13 (redwoodjs#7915)
  chore(deps): update dependency lerna to v6.6.1 (redwoodjs#7916)
  chore(deps): update dependency @tsd/typescript to v5 (redwoodjs#7856)
  chore(deps): update dependency @testing-library/dom to v9.1.0 (redwoodjs#7914)
  fix(deps): update dependency msw to v1.2.1 (redwoodjs#7911)
  chore(deps): update dependency @types/react to v18.0.29 (redwoodjs#7912)
  chore(deps): update dependency nx to v15.8.9 (redwoodjs#7913)
  Fix for directUrl usage during testing (redwoodjs#7898)
  fix(deps): update dependency prettier to v2.8.7 (redwoodjs#7908)
  chore(deps): update dependency @nrwl/nx-cloud to v15.3.2 (redwoodjs#7904)
  fix(deps): update dependency @types/node to v16.18.19 (redwoodjs#7906)
  ...
@jtoar jtoar modified the milestones: next-release, v4.5.0 Apr 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release:chore This PR is a chore (means nothing for users)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants