perf(linter/no-unused-vars): use default IgnorePattern when /^_/ is provided as a pattern

[DonIsaac]

Use IgnorePattern::Default even when /^_/ is explicitly provided as a regex. This gives better diagnostic messages and lets us use starts_with('_') in more cases.

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[DonIsaac]

• perf(linter/no-unused-vars): use default IgnorePattern when /^_/ is provided as a pattern #6697 👈
• feat(linter/eslint): show ignore patterns in eslint/no-unused-vars diagnostic messages #6696
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #6697 will not alter performance

_{Comparing don/10-19-perf_linter/no-unused-vars_use_default_ignorepattern_when_/__/_is_provided_as_a_pattern (159fe13) with main (85e69a1)}

Summary

✅ 30 untouched benchmarks

[DonIsaac]

Merge activity

• Oct 20, 12:24 AM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Oct 20, 12:28 AM EDT: A user added this pull request to the Graphite merge queue.
• Oct 20, 12:28 AM EDT: The Graphite merge queue couldn't merge this PR because it had conflicts with the trunk branch.