Bump the eslint group with 4 updates #2176
Merged
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dependabot
bot
added
dependencies
dependabot
bot
force-pushed
the
dependabot/npm_and_yarn/eslint-18dfe0ba04
branch
7 times, most recently
from
ee2a90b to
c2c59bf
Compare
slifty
force-pushed
the
dependabot/npm_and_yarn/eslint-18dfe0ba04
branch
from
c2c59bf to
6e489ca
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2176 +/- ##
=======================================
Coverage 93.21% 93.21%
=======================================
Files 324 324
Lines 3772 3772
Branches 558 557 -1
=======================================
Hits 3516 3516
Misses 256 256 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
bickelj
approved these changes
Jan 2, 2026
Contributor
bickelj
left a comment
bickelj
left a comment
There was a problem hiding this comment.
TIL regex non-capturing groups and named capture groups.
I consulted an LLM, the docs, and a search to understand.
Here is a handy table Qwen3 produced when explaining the non-named groups:
| Feature | Original Regex (() ) |
Changed Regex ((?:) ) |
|---|---|---|
| Capturing Groups | Yes (groups 1 and 2) | No (groups are non-capturing) |
| Performance | Slightly slower (capturing groups require memory) | Slightly faster (non-capturing groups are lighter) |
| Group Indexing | Groups are numbered (e.g., match[1] for fractional seconds) |
Groups are not numbered (no match[1] or match[2]) |
| Use Case | Required if you need to extract parts of the match | Preferred if you don’t need to extract parts |
And of course the rule details at https://eslint.org/docs/latest/rules/prefer-named-capture-group made it clear.
Since I hadn't seen named groups before I was curious what regular-expressions.info had to say, and it says Python pioneered it and .NET made their own flavor https://www.regular-expressions.info/named.html.
Now it makes sense and is indeed a good rule!
The tests pass, so I don't feel the need to look any deeper.
Bumps the eslint group with 4 updates: [@eslint/js](https://github.com/eslint/eslint/tree/HEAD/packages/js), [eslint](https://github.com/eslint/eslint), [eslint-config-love](https://github.com/mightyiam/eslint-config-love) and [typescript-eslint](https://github.com/typescript-eslint/typescript-eslint/tree/HEAD/packages/typescript-eslint). Updates `@eslint/js` from 9.39.1 to 9.39.2 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](https://github.com/eslint/eslint/commits/v9.39.2/packages/js) Updates `eslint` from 9.39.1 to 9.39.2 - [Release notes](https://github.com/eslint/eslint/releases) - [Commits](eslint/eslint@v9.39.1...v9.39.2) Updates `eslint-config-love` from 136.0.0 to 140.0.0 - [Release notes](https://github.com/mightyiam/eslint-config-love/releases) - [Changelog](https://github.com/mightyiam/eslint-config-love/blob/master/CHANGELOG.md) - [Commits](mightyiam/eslint-config-love@v136.0.0...v140.0.0) Updates `typescript-eslint` from 8.48.1 to 8.50.1 - [Release notes](https://github.com/typescript-eslint/typescript-eslint/releases) - [Changelog](https://github.com/typescript-eslint/typescript-eslint/blob/main/packages/typescript-eslint/CHANGELOG.md) - [Commits](https://github.com/typescript-eslint/typescript-eslint/commits/v8.50.1/packages/typescript-eslint) --- updated-dependencies: - dependency-name: "@eslint/js" dependency-version: 9.39.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: eslint - dependency-name: eslint dependency-version: 9.39.2 dependency-type: direct:development update-type: version-update:semver-patch dependency-group: eslint - dependency-name: eslint-config-love dependency-version: 140.0.0 dependency-type: direct:development update-type: version-update:semver-major dependency-group: eslint - dependency-name: typescript-eslint dependency-version: 8.50.1 dependency-type: direct:development update-type: version-update:semver-minor dependency-group: eslint ... Signed-off-by: dependabot[bot] <support@github.com>
slifty
force-pushed
the
dependabot/npm_and_yarn/eslint-18dfe0ba04
branch
from
6e489ca to
e4a85d5
Compare
github-actions
bot
deleted the
dependabot/npm_and_yarn/eslint-18dfe0ba04
branch
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bumps the eslint group with 4 updates: @eslint/js, eslint, eslint-config-love and typescript-eslint.
Updates
@eslint/jsfrom 9.39.1 to 9.39.2Release notes
Sourced from
@eslint/js's releases.Commits
c43ce24chore: package.json update for@eslint/jsreleaseUpdates
eslintfrom 9.39.1 to 9.39.2Release notes
Sourced from eslint's releases.
Commits
92783249.39.2542266aBuild: changelog update for 9.39.27ca0af7chore: upgrade to@eslint/js@9.39.2(#20394)c43ce24chore: package.json update for@eslint/jsrelease5705833fix: warn wheneslint-envconfiguration comments are found (#20381)506f154build: add .scss files entry to knip (#20391)4c9858eci: addv9.x-devbranch (#20382)Updates
eslint-config-lovefrom 136.0.0 to 140.0.0Release notes
Sourced from eslint-config-love's releases.
... (truncated)
Changelog
Sourced from eslint-config-love's changelog.
Commits
c52d2edchore(release): 140.0.0 [skip ci]1274725feat!: prefer-object-spread1f98a70chore(release): 139.0.0 [skip ci]3e9ab57feat!: prefer-object-has-own00d8b48chore(release): 138.0.1 [skip ci]0e71979ci(publish): var and secret namese0bb16aRevert "ci: rm environment from publish job"0e294f8chore(release): 138.0.0 [skip ci]ee234c1ci: publish job GH_TOKEN -> GITHUB_TOKENd411ce5ci: rm environment from publish jobMaintainer changes
This version was pushed to npm by [GitHub Actions](https://www.npmjs.com/~GitHub Actions), a new releaser for eslint-config-love since your current version.
Updates
typescript-eslintfrom 8.48.1 to 8.50.1Release notes
Sourced from typescript-eslint's releases.
Changelog
Sourced from typescript-eslint's changelog.
Commits
d520b88chore(release): publish 8.50.1c62e858chore(release): publish 8.50.0864595achore(release): publish 8.49.032b7e89chore(deps): update dependency@vitest/eslint-pluginto v1.5.1 (#11816)Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting
@dependabot rebase.Dependabot commands and options
You can trigger Dependabot actions by commenting on this PR:
@dependabot rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot mergewill merge this PR after your CI passes on it@dependabot squash and mergewill squash and merge this PR after your CI passes on it@dependabot cancel mergewill cancel a previously requested merge and block automerging@dependabot reopenwill reopen this PR if it is closed@dependabot closewill close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditions