Make argsIgnorePattern for no-unused-vars lint rule more strict

[dario-piotrowicz]

Right now we completely ignore unused vars when they are arguments of functions, I am not sure if this is intentional, wouldn't it be better/cleaner to have lint error on those as we do for other variables? (and ignore args that start with _, allowing us to be explicit on when we do want to have unused arguments)

---

• Tests
  • Tests included/updated
  • Automated tests not possible - manual testing has been completed as follows:
  • Additional testing not necessary because:
• Public documentation
  • Cloudflare docs PR(s):
  • Documentation not necessary because: internal refactoring

A picture of a cute animal (not mandatory, but encouraged)

---

[changeset-bot]

⚠️ No Changeset found

Latest commit: 8d38e09

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[ask-bonk]

UnknownError: ProviderInitError

github run

[ask-bonk]

@dario-piotrowicz Bonk workflow failed. Check the logs for details.

View workflow run · To retry, trigger Bonk again.

[workers-devprod]

Codeowners approval required for this PR:

• @cloudflare/deploy-config
• @cloudflare/workers-kv
• ✅ @cloudflare/wrangler

Show detailed file reviewers

• packages/miniflare/test/plugins/kv/index.spec.ts: [@cloudflare/workers-kv]
• packages/workers-shared/router-worker/tests/index.test.ts: [@cloudflare/deploy-config]

[pkg-pr-new]

create-cloudflare

npm i https://pkg.pr.new/create-cloudflare@13180

@cloudflare/kv-asset-handler

npm i https://pkg.pr.new/@cloudflare/kv-asset-handler@13180

miniflare

npm i https://pkg.pr.new/miniflare@13180

@cloudflare/pages-shared

npm i https://pkg.pr.new/@cloudflare/pages-shared@13180

@cloudflare/unenv-preset

npm i https://pkg.pr.new/@cloudflare/unenv-preset@13180

@cloudflare/vite-plugin

npm i https://pkg.pr.new/@cloudflare/vite-plugin@13180

@cloudflare/vitest-pool-workers

npm i https://pkg.pr.new/@cloudflare/vitest-pool-workers@13180

@cloudflare/workers-editor-shared

npm i https://pkg.pr.new/@cloudflare/workers-editor-shared@13180

wrangler

npm i https://pkg.pr.new/wrangler@13180

commit: 8d38e09

[workers-devprod]

Codeowners Bypass