Skip to content

[v2] core: clone service type aliases instead of referencing global slice - #3473

Merged
kayrus merged 1 commit into
v2from
bp-v2-9e5efb7
Jul 22, 2025
Merged

kayrus merged 1 commit into
v2from
bp-v2-9e5efb7

Conversation

@gophercloud-backport-bot

Copy link
Copy Markdown

Backport: #3462

Without this fix modifying the eo.Aliases causes the modification of the global map slice entry.

We might need to enable linter that catches this.

UPD: gocritic doesn't catch this case :(

@coveralls

Copy link
Copy Markdown

Coverage Status

coverage: 63.637%. remained the same
when pulling fcf7c1d on bp-v2-9e5efb7
into 015f396 on v2.

@github-actions github-actions Bot added semver:patch No API change v2 This PR targets v2 edit:gophercloud This PR updates common Gophercloud code labels Jul 21, 2025
@kayrus
kayrus merged commit d3a1523 into v2 Jul 22, 2025
@kayrus
kayrus deleted the bp-v2-9e5efb7 branch July 22, 2025 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

edit:gophercloud This PR updates common Gophercloud code semver:patch No API change v2 This PR targets v2

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants