fix: fix internal package name regex and field and remove package groups #38

Merged
t.behrendt merged 1 commits from fix-exclude-internal-packages-from-min-release-age into main 2026-02-14 12:57:41 +01:00
Owner

Dropping the "matchManagers" as we actually want to match anything, regardless of manager.
Also fixing the regex, wasn't valid before.
Also renaming the "matchPackagePatterns" to "matchPackageNames" as the previous value isn't supported anymore..

Since it's JSON, renovate needs the regex to be quoted. See example:

{
  "packageRules": [
    {
      "matchDatasources": ["npm"],
      "matchPackageNames": ["/^angular/"],
      "groupName": "Angular"
    }
  ]
}
Dropping the "matchManagers" as we actually want to match anything, regardless of manager. Also fixing the regex, wasn't valid before. Also renaming the "matchPackagePatterns" to "matchPackageNames" as the previous value isn't supported anymore.. Since it's JSON, renovate needs the regex to be quoted. See example: ```json { "packageRules": [ { "matchDatasources": ["npm"], "matchPackageNames": ["/^angular/"], "groupName": "Angular" } ] } ```
t.behrendt added 1 commit 2026-02-14 12:47:38 +01:00
fix: fix internal package name regex and field and remove package groups
All checks were successful
CI / Test (action.json) (pull_request) Successful in 14s
CI / Test (helm.json) (pull_request) Successful in 13s
CI / Test (k8s.json) (pull_request) Successful in 45s
CI / Test (common.json) (pull_request) Successful in 48s
CI / Test (docker-compose.json) (pull_request) Successful in 1m2s
b1cd7c0dc7
t.behrendt requested review from branch-buddy 2026-02-14 12:49:13 +01:00
branch-buddy approved these changes 2026-02-14 12:53:07 +01:00
branch-buddy left a comment
Collaborator

The regex pattern in matchPackageNames appears to have syntax issues. The pattern starts with / but is enclosed in quotes, which may not function as intended. Also, the trailing / in the original matchPackagePatterns may be incorrect.

The regex pattern in matchPackageNames appears to have syntax issues. The pattern starts with / but is enclosed in quotes, which may not function as intended. Also, the trailing / in the original matchPackagePatterns may be incorrect.
Collaborator

Verify the regex syntax for matchPackageNames to ensure it correctly matches intended patterns.

Verify the regex syntax for matchPackageNames to ensure it correctly matches intended patterns.
Collaborator

Check if the trailing / in the original matchPackagePatterns was intentional.

Check if the trailing / in the original matchPackagePatterns was intentional.
Collaborator

Ensure the renaming from matchPackagePatterns to matchPackageNames aligns with the new logic.

Ensure the renaming from matchPackagePatterns to matchPackageNames aligns with the new logic.
Collaborator

Verify that all instances of "matchManagers" are removed from related configurations

Verify that all instances of "matchManagers" are removed from related configurations
Collaborator

Ensure the regex pattern continues to correctly match intended package names

Ensure the regex pattern continues to correctly match intended package names
Collaborator

Confirm that the JSON structure remains valid across all affected files

Confirm that the JSON structure remains valid across all affected files
t.behrendt marked this conversation as resolved
t.behrendt removed review request for branch-buddy 2026-02-14 12:56:24 +01:00
branch-buddy approved these changes 2026-02-14 12:57:29 +01:00
branch-buddy left a comment
Collaborator

The changes align with the PR title and description. The regex pattern was corrected to start with /, and "matchManagers" was removed as specified. The JSON syntax is valid and matches the provided example.

The changes align with the PR title and description. The regex pattern was corrected to start with /, and "matchManagers" was removed as specified. The JSON syntax is valid and matches the provided example.
Collaborator

Verify the regex syntax for matchPackageNames to ensure it correctly matches intended patterns.

Verify the regex syntax for matchPackageNames to ensure it correctly matches intended patterns.
Collaborator

Check if the trailing / in the original matchPackagePatterns was intentional.

Check if the trailing / in the original matchPackagePatterns was intentional.
Collaborator

Ensure the renaming from matchPackagePatterns to matchPackageNames aligns with the new logic.

Ensure the renaming from matchPackagePatterns to matchPackageNames aligns with the new logic.
Collaborator

Verify that all instances of "matchManagers" are removed from related configurations

Verify that all instances of "matchManagers" are removed from related configurations
Collaborator

Ensure the regex pattern continues to correctly match intended package names

Ensure the regex pattern continues to correctly match intended package names
Collaborator

Confirm that the JSON structure remains valid across all affected files

Confirm that the JSON structure remains valid across all affected files
t.behrendt marked this conversation as resolved
t.behrendt merged commit e8e303565a into main 2026-02-14 12:57:41 +01:00
t.behrendt deleted branch fix-exclude-internal-packages-from-min-release-age 2026-02-14 12:57:41 +01:00
Sign in to join this conversation.
No Reviewers
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: t.behrendt/renovate-configs#38