Skip to content

Conversation

runningcode
Copy link
Contributor

@runningcode runningcode commented Sep 29, 2025

Summary

Fixes EME-362 by reducing excessive debug logging when fetching PR numbers during build upload commands.

Changes

  • Reduced debug logging in get_github_pr_number() from 4 statements to 2 essential ones
  • Removed intermediate debug messages: "Extracted PR reference" and "Parsing PR number from"
  • Kept the most important debug logs: event type validation and final PR number detection

Before

debug!("Not running in pull_request event, got: {}", event_name);
debug!("Extracted PR reference: {}", pr_number_str);
debug!("Parsing PR number from: {}", pr_number_str); 
debug!("Auto-detected PR number from GitHub Actions: {}", pr_number);

After

debug!("Not running in pull_request event, got: {}", event_name);
debug!("Auto-detected PR number from GitHub Actions: {}", pr_number);

🤖 Generated with Claude Code


Note

Remove two intermediate debug logs in get_github_pr_number(), keeping only event validation and final PR number detection.

Written by Cursor Bugbot for commit a2ed966. This will update automatically on new commits. Configure here.

Reduced excessive debug logging in get_github_pr_number() from 4 debug
statements to 2 essential ones. This removes the intermediate "Extracted PR
reference" and "Parsing PR number from" messages while keeping the most
important logs for debugging: event type mismatch and final PR number
detection.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <[email protected]>
Copy link

linear bot commented Sep 29, 2025

@runningcode runningcode marked this pull request as ready for review September 29, 2025 14:41
@runningcode runningcode requested review from a team and szokeasaurusrex as code owners September 29, 2025 14:41
@runningcode runningcode enabled auto-merge (squash) September 29, 2025 14:43
@runningcode runningcode merged commit 5b1fb22 into master Sep 29, 2025
25 checks passed
@runningcode runningcode deleted the fix/eme-362-simplify-pr-debug-logging branch September 29, 2025 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants