Skip to content

ref(seer): Move get_github_username_for_user to shared utils#112669

Merged
JoshFerge merged 1 commit intomasterfrom
joshferge/ref/shared-github-username-util
Apr 10, 2026
Merged

ref(seer): Move get_github_username_for_user to shared utils#112669
JoshFerge merged 1 commit intomasterfrom
joshferge/ref/shared-github-username-util

Conversation

@JoshFerge
Copy link
Copy Markdown
Member

@JoshFerge JoshFerge commented Apr 10, 2026

Summary

Extract get_github_username_for_user from autofix.py to seer/utils.py so it can be reused by the explorer flow.

No behavior change.

Stack

Test plan

  • All 13 TestGetGithubUsernameForUser tests pass
  • TestCallAutofix passes

Agent transcript: https://claudescope.sentry.dev/share/gQE_FjdkKYG_ni3m7GJV0vvaMby0_VYjfOfMHJCpAbA

@github-actions github-actions bot added the Scope: Backend Automatically applied to PRs that change backend components label Apr 10, 2026
@JoshFerge JoshFerge force-pushed the joshferge/ref/shared-github-username-util branch from 8a6d955 to 950be54 Compare April 10, 2026 14:39
@JoshFerge JoshFerge force-pushed the joshferge/ref/shared-github-username-util branch from 950be54 to 5a0d4bc Compare April 10, 2026 14:42
@JoshFerge JoshFerge marked this pull request as ready for review April 10, 2026 14:44
@JoshFerge JoshFerge requested a review from a team as a code owner April 10, 2026 14:44
Copy link
Copy Markdown
Contributor

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 5a0d4bc. Configure here.

Comment thread tests/sentry/seer/autofix/test_autofix.py Outdated
Extract the GitHub username resolution function from autofix.py to
seer/utils.py so it can be reused by the explorer flow. The old name
is re-exported from autofix.py for backwards compatibility.

No behavior change.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Agent transcript: https://claudescope.sentry.dev/share/8BmYM4cKcaUl38fiIA-xxoHkUFgXmsXGQ2qrpAgJuKs
Copy link
Copy Markdown
Contributor

@chromy chromy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@JoshFerge JoshFerge enabled auto-merge (squash) April 10, 2026 15:06
@JoshFerge JoshFerge merged commit ebed79d into master Apr 10, 2026
57 checks passed
@JoshFerge JoshFerge deleted the joshferge/ref/shared-github-username-util branch April 10, 2026 15:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants