fix: patch grouping when separateMinorPatch=false

[rarkins]

Changes:

Fixes logic so that patch updates don't separate themselves from groupings.

Context:

Previously, if updateType=patch then it could only be because separateMinorPatch=true, but now that is not the case and the extra checks are required.

Documentation (please check one with an [x])

• I have updated the documentation, or
• No documentation update is required

How I've tested my work (please tick one)

I have verified these changes via:

• Code inspection only, or
• Newly added unit tests, or
• No new tests but ran on a real repository, or
• Both unit tests + ran on a real repository

[renovate-release]

🎉 This PR is included in version 25.4.3 🎉

The release is available on:

• GitHub release
• 25.4.3

Your semantic-release bot 📦🚀

[hisener]

@rarkins Maybe a stupid question but why don't we have update.updateType === 'minor' check here?

We use 6.1.7.Final of hibernate-validator and there is only one upgrade PR from 6.1.7.Final to 7.0.1.Final even though the configuration is:

{
  "packagePatterns": [
    "^version\\.hibernate-validator$",
  ],
  "separateMajorMinor": true,
  "separateMinorPatch": true
}

It could also because of grouping or .Final at the end, not sure. Does this ring a bell?

[rarkins]

Can you create a reproduction repo to demonstrate and create a new discussion?

[hisener]

Update: I couldn't reproduce it in a public repo. There two PRs as expected: hisener/renovate-tests#60 and hisener/renovate-tests#61.

First, I will upgrade the Renovate version. Probably that's the issue.

[hisener]

Another update: Filed #10190. :)