Skip to content

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jun 3, 2024

Mend Renovate

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
eslint (source) 8.57.0 -> 9.7.0 age adoption passing confidence

Release Notes

eslint/eslint (eslint)

v9.7.0

Compare Source

Features

  • 7bd9839 feat: add support for es2025 duplicate named capturing groups (#​18630) (Yosuke Ota)
  • 1381394 feat: add regex option in no-restricted-imports (#​18622) (Nitin Kumar)

Bug Fixes

  • 14e9f81 fix: destructuring in catch clause in no-unused-vars (#​18636) (Francesco Trotta)

Documentation

  • 9f416db docs: Add Powered by Algolia label to the search. (#​18633) (Amaresh S M)
  • c8d26cb docs: Open JS Foundation -> OpenJS Foundation (#​18649) (Milos Djermanovic)
  • 6e79ac7 docs: loadESLint does not support option cwd (#​18641) (Francesco Trotta)

Chores

v9.6.0

Compare Source

v9.5.0

Compare Source

v9.4.0

Compare Source

v9.3.0

Compare Source

v9.2.0

Compare Source

v9.1.1

Compare Source

v9.1.0

Compare Source

v9.0.0

Compare Source


Configuration

📅 Schedule: Branch creation - "on Monday every 9 weeks of the year starting on the 5th week" (UTC), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@renovate renovate bot requested a review from a team as a code owner June 3, 2024 06:45
@renovate renovate bot force-pushed the renovate/major-eslint-monorepo branch 9 times, most recently from ddbcff8 to 84e999e Compare June 6, 2024 16:52
@renovate renovate bot force-pushed the renovate/major-eslint-monorepo branch 2 times, most recently from 0739b59 to d5f2cca Compare June 18, 2024 19:40
@renovate renovate bot force-pushed the renovate/major-eslint-monorepo branch 3 times, most recently from a8d6234 to f345f52 Compare July 1, 2024 19:31
@renovate renovate bot force-pushed the renovate/major-eslint-monorepo branch 4 times, most recently from 3ead230 to 9fbd22c Compare July 16, 2024 17:50
@geotrev
Copy link
Contributor

geotrev commented Jul 17, 2024

Waiting for update to CI to allow bot builds.

@ze-flo
Copy link
Contributor

ze-flo commented Jul 17, 2024

Waiting for update to CI to allow bot builds.

The linting test is failing but there's no indication it's related to the CI config. Is it passing locally?

@jzempel
Copy link
Member

jzempel commented Jul 18, 2024

Waiting for update to CI to allow bot builds.

The linting test is failing but there's no indication it's related to the CI config. Is it passing locally?

This repo hasn't yet been configured for eslint v9 😉

@geotrev
Copy link
Contributor

geotrev commented Jul 24, 2024

Waiting for update to CI to allow bot builds.

The linting test is failing but there's no indication it's related to the CI config. Is it passing locally?

yes, this is true, good point. haha also the lack of eslint v9 config. 🤦🏻

Copy link
Contributor Author

renovate bot commented Jul 26, 2024

Edited/Blocked Notification

Renovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR.

You can manually request rebase by checking the rebase/retry box above.

⚠️ Warning: custom changes will be lost.

@geotrev
Copy link
Contributor

geotrev commented Jul 30, 2024

Status of the update:

  • Still encountering issues with test files. Jest globals don't seem to be recognized like they should (see react-containers where an identical jestPlugin implementation works as expected). Attempting to re-apply jestPlugin.plugins doesn't seem to have any effect here.
  • garden-local works but the linter runs five times slower. When attempting to run with typescriptPlugin spread to the config entry, typescript complains about @typescript-eslint/adjacent-overload-signatures. Instead, I tried to spread in typescriptPlugin.plugins which ends up stalling eslint indefinitely.

Using eslint's handy-dandy inspector, I can see rules are being applied, but clearly the actual linting process is still kinda jank. 💀

@jzempel
Copy link
Member

jzempel commented Aug 21, 2024

Jest globals don't seem to be recognized like they should

@geotrev ha, those typography lists test files still had .spec.js extensions rather than .spec.tsx

package.json Outdated
"@typescript-eslint/parser": "7.18.0",
"@zendeskgarden/css-bedrock": "9.1.1",
"@zendeskgarden/eslint-config": "40.0.0",
"@zendeskgarden/eslint-config": "42.0.0",
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we bump this to the latest v43.0.0 instead?


return (
<StyledCalendarItem key={`day-${itemsIndex}`} $isCompact={isCompact}>
<StyledCalendarItem key={date.toISOString()} $isCompact={isCompact}>
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

Copy link
Contributor

@ze-flo ze-flo left a comment

Choose a reason for hiding this comment

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

Great work!

@jzempel jzempel merged commit e009570 into main Aug 22, 2024
@jzempel jzempel deleted the renovate/major-eslint-monorepo branch August 22, 2024 14:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

3 participants