Skip to content

refactor: remove duplicated code - #1654

Merged
mbrandenburger merged 1 commit into
mainfrom
ai-findings-autofix/platform-view-services-id-ecdsa-view.go
Aug 18, 2026
Merged

refactor: remove duplicated code#1654
mbrandenburger merged 1 commit into
mainfrom
ai-findings-autofix/platform-view-services-id-ecdsa-view.go

Conversation

@mbrandenburger

@mbrandenburger mbrandenburger commented Aug 18, 2026

Copy link
Copy Markdown
Member

This PR applies 1/1 suggestions from code quality AI findings.

@mbrandenburger mbrandenburger changed the title Potential fix for 1 code quality finding refactor: remove duplicated code Aug 18, 2026
@mbrandenburger mbrandenburger added the cleanup Something needs some cleanup label Aug 18, 2026
@mbrandenburger mbrandenburger added this to the 26Q3 milestone Aug 18, 2026
@mbrandenburger
mbrandenburger marked this pull request as ready for review August 18, 2026 08:15
@github-actions

github-actions Bot commented Aug 18, 2026 •
edited
Loading

Copy link
Copy Markdown

Hey @mbrandenburger 👋 thanks for the PR!
I'm your friendly PR Helper Bot 🤖 and I'll be riding shotgun on this one, keeping track of your PR's status to help you get it approved and merged.

This comment updates automatically as you push changes -- think of it as your PR's live scoreboard!
Here's the latest:

PR Checks

✅ DCO Sign-off -- All commits have valid sign-offs. Nice work!

❌ GPG Signature -- Heads up! The following commits don't have a verified GPG signature:

  • 9fd36bd Apply suggested fix to platform/view/services/id/ecdsa/view.go from Copilot Autofix

You'll need to sign your commits with GPG (e.g. git commit -S). See the Signing Guide for a step-by-step walkthrough.

✅ Merge Conflicts -- No merge conflicts detected. Smooth sailing!

❌ Issue Link -- This PR is not linked to any issue.

Please reference an issue using a closing keyword (e.g. Fixes #123) and ensure the issue is assigned to you. Every PR needs a home!

⏳ All checks must pass before this PR can be reviewed. You've got this!

@github-actions github-actions Bot added the status: needs revision The pull request requires changes from the author before it can be reviewed or merged label Aug 18, 2026
@mergify

mergify Bot commented Aug 18, 2026

Copy link
Copy Markdown

Tick the box to add this pull request to the merge queue (same as @mergifyio queue).

  • Queue this pull request

@mbrandenburger
mbrandenburger enabled auto-merge (squash) August 18, 2026 08:24
@mbrandenburger @github-code-quality
Apply suggested fix to platform/view/services/id/ecdsa/view.go from C…
9fd36bd 
…opilot Autofix

Co-authored-by: Copilot Autofix powered by AI <223894421+github-code-quality[bot]@users.noreply.github.com>
Signed-off-by: Marcus Brandenburger <mbrandenburger@users.noreply.github.com>
@mbrandenburger
mbrandenburger force-pushed the ai-findings-autofix/platform-view-services-id-ecdsa-view.go branch from 958c18c to 9fd36bd Compare August 18, 2026 08:24
@mbrandenburger
mbrandenburger merged commit 9ef3fe3 into main Aug 18, 2026
11 of 12 checks passed
@mbrandenburger
mbrandenburger deleted the ai-findings-autofix/platform-view-services-id-ecdsa-view.go branch August 18, 2026 08:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@SaidAltury-ibm SaidAltury-ibm SaidAltury-ibm approved these changes

Assignees

@mbrandenburger mbrandenburger

Labels

cleanup Something needs some cleanup status: needs revision The pull request requires changes from the author before it can be reviewed or merged

Projects

None yet

Milestone

26Q3

Development

Successfully merging this pull request may close these issues.

2 participants

@mbrandenburger @SaidAltury-ibm