feat: sharedVariableName

[rarkins]

Changes

Replaces groupName in manager extract results with sharedVariableName instead. I chose to keep groupName later for simplicity and backwards compatibility.

Context

We should still group updates if they have shared variables, but separate the concept from manually-configured groupNames.

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 select one)

I have verified these changes via:

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

[rarkins]

Needs testing on at least one real repo

[rarkins]

FYI I want to have a similar concept for monorepoName after this

[secustor]

We should add docs to https://github.com/renovatebot/renovate/blob/main/docs%2Fdevelopment%2Fadding-a-package-manager.md too

[rarkins]

I think let's wait until the new year to merge this one, as it could be disruptive if wrong

[rarkins]

Let's merge this after Jan 6

[rarkins]

@zharinov could you look for at least one "real" repo to test against? You might still have the ones you tested against back when you added support for variables

[Churro]

Tested successfully for gradle using randomly picked old reproduction repos that use variables:

• https://github.com/renovate-demo/18317-renovate-gradle-variable-interpolate-reproduction2
• https://github.com/renovate-demo/25373-renovate-gradle-issue2

[Churro]

Unrelated to this change but I think this no longer needed:

renovate/lib/modules/manager/types.ts

Line 170 in c47224e

depGroup?: string;

[rarkins]

Unrelated to this change but I think this no longer needed:

renovate/lib/modules/manager/types.ts

Line 170 in c47224e

depGroup?: string;

I'd forgotten about this, but this was an earlier attempt I made to fix the same problem

[zharinov]

@zharinov could you look for at least one "real" repo to test against? You might still have the ones you tested against back when you added support for variables

Just updated this one: https://github.com/renovate-testing/test-10436-lein-vars