Skip to content

Conversation

merceyz
Copy link
Member

@merceyz merceyz commented Jul 14, 2022

What's the problem this PR addresses?

Node.js v18.6 requires that loaders explicitly return shortCircuit: true when they don't call the next loader in the chain.

How did you fix it?

Update the hooks.

Checklist

  • I have read the Contributing Guide.
  • I have set the packages that need to be released for my changes to be effective.
  • I will check that all automated PR checks pass before the PR gets reviewed.

@arcanis arcanis merged commit 4ed2ef8 into master Jul 14, 2022
@arcanis arcanis deleted the merceyz/fix/pnp-node-chaining branch July 14, 2022 13:52
tumidi added a commit to innodoc/innodoc-webapp that referenced this pull request Feb 29, 2024
  - Modularize
  - Add workaround for eslint/eslint#3458
    https://github.com/microsoft/rushstack/blob/main/eslint/eslint-config/patch/modern-module-resolution.js
- Enable pnp esm loader
- Project directory restructure
    /packages/client-web/e2e -> /e2e
    /packages/client-web -> /app
    rename packages/*
- Use yarn canary until yarnpkg/berry#4624 is released.
- Remove old tests (must be ported to RTL)
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