Skip to content

Conversation

@MikeMcC399
Copy link
Collaborator

Issue

ESlint reports warnings from an externally sourced Yarn Plug'n'Play module:

github-action/examples/yarn-modern-pnp/.pnp.cjs
  8519:4   warning  Unused eslint-disable directive (no problems were reported from 'operator-linebreak')
  8570:10  warning  Unused eslint-disable directive (no problems were reported from 'no-undef')

Change

Add externally sourced Yarn Plug'n'Play modules to global ignore list of eslint.config.mjs

examples/**/.pnp.*

Verification

Ubuntu 22.04.2 LTS, Node.js 22.14.0 LTS

git clean -xfd
npm ci
corepack enable yarn
cd examples/yarn-modern-pnp
yarn
cd ../..
npx eslint

and confirm no ESLint warnings or errors are shown.

@MikeMcC399 MikeMcC399 added the bug Something isn't working label Apr 28, 2025
@MikeMcC399 MikeMcC399 self-assigned this Apr 28, 2025
@cypress-app-bot
Copy link

@MikeMcC399 MikeMcC399 marked this pull request as ready for review April 28, 2025 11:27
@jennifer-shehane jennifer-shehane merged commit 6f72d0f into cypress-io:master Apr 28, 2025
74 checks passed
@MikeMcC399 MikeMcC399 deleted the eslint-ignore-yarn-pnp branch April 28, 2025 14:13
@github-actions
Copy link

🎉 This PR is included in version 6.8.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working released

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants