Skip to content

refactor: Trivial fixes#3515

Merged
pierreprinetti merged 1 commit intogophercloud:mainfrom
shiftstack:trivial
Sep 22, 2025
Merged

refactor: Trivial fixes#3515
pierreprinetti merged 1 commit intogophercloud:mainfrom
shiftstack:trivial

Conversation

@pierreprinetti
Copy link
Copy Markdown
Member

Address some gopls suggestions. The behaviour of the code is expected to remain unchanged.

  • That docs.go file is a noop
  • The use of slices in provider_client.go makes the code easier to read
  • The err check I remove happened where err is untouched after the previous check.

Address some gopls suggestions. The behaviour of the code is expected to
remain unchanged.
@github-actions github-actions bot added edit:gophercloud This PR updates common Gophercloud code semver:patch No API change backport-v2 This PR will be backported to v2 labels Sep 21, 2025
@coveralls
Copy link
Copy Markdown

Coverage Status

coverage: 63.808% (-0.002%) from 63.81%
when pulling 8397c0e on shiftstack:trivial
into 217eaa1 on gophercloud:main.

@pierreprinetti pierreprinetti self-assigned this Sep 21, 2025
@pierreprinetti pierreprinetti requested a review from a team September 21, 2025 14:08
@stephenfin stephenfin added this pull request to the merge queue Sep 22, 2025
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Sep 22, 2025
@pierreprinetti pierreprinetti added this pull request to the merge queue Sep 22, 2025
Merged via the queue into gophercloud:main with commit 99dfe55 Sep 22, 2025
80 checks passed
@pierreprinetti pierreprinetti deleted the trivial branch September 22, 2025 13:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-v2 This PR will be backported to v2 edit:gophercloud This PR updates common Gophercloud code semver:patch No API change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants