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

Bump express-validator from 6.14.3 to 6.15.0 #3304

Merged
merged 2 commits into from
Feb 21, 2023

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Feb 17, 2023

Bumps express-validator from 6.14.3 to 6.15.0.

Release notes

Sourced from express-validator's releases.

v6.15.0

What's Changed

Full Changelog: express-validator/express-validator@v6.14.3...v6.15.0

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.


Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

Bumps [express-validator](https://github.com/express-validator/express-validator) from 6.14.3 to 6.15.0.
- [Release notes](https://github.com/express-validator/express-validator/releases)
- [Commits](express-validator/express-validator@v6.14.3...v6.15.0)

---
updated-dependencies:
- dependency-name: express-validator
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>
@dependabot dependabot bot added dependencies Pull requests that update a dependency file javascript labels Feb 17, 2023
@dependabot dependabot bot requested a review from a team February 17, 2023 09:59
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3304 February 17, 2023 09:59 Inactive
@colinrotherham
Copy link
Contributor

colinrotherham commented Feb 17, 2023

Likely due to npm package incompatibilities, this update can't be hoisted to node_modules

Checks have failed since we cache node_modules but not app/node_modules

1. express-validator in node_modules app/node_modules

└─┬ govuk-frontend-review@ -> ./app
  └── express-validator@6.15.0

2. validator in node_modules app/node_modules

└─┬ govuk-frontend-review@ -> ./app
  └─┬ express-validator@6.15.0
    └── validator@13.9.0

I'll update the cache paths to make sure app/node_modules are included

To avoid package incompatibilities, npm can choose _not_ to hoist to the top-level `node_modules` directory

We have to avoid matching `**/node_modules` otherwise nested “packages of packages” will be cached from linked workspaces etc
@govuk-design-system-ci govuk-design-system-ci temporarily deployed to govuk-frontend-pr-3304 February 17, 2023 13:34 Inactive
path: node_modules
path: |
node_modules
app/node_modules
Copy link
Contributor

Choose a reason for hiding this comment

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

We should ensure any (potential) workspace node_modules are also cached

Due to symbolic links created by npm link or npm workspaces we can't use:

path: |
  **/node_modules

Otherwise we could match and cache "packages of linked packages" one day:

app/node_modules/govuk-frontend/node_modules/** #symlink-1
node_modules/govuk-frontend/node_modules/** #symlink-1
package/node_modules/** #source

@colinrotherham colinrotherham self-requested a review February 17, 2023 13:46
Copy link
Member

@romaricpascal romaricpascal left a comment

Choose a reason for hiding this comment

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

Thanks for sorting the caching 🙌🏻

@romaricpascal romaricpascal merged commit 7a3df8e into main Feb 21, 2023
@romaricpascal romaricpascal deleted the dependabot/npm_and_yarn/express-validator-6.15.0 branch February 21, 2023 09:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants