Skip to content

actions, pull_request: Check reward weight correctly on outdated branch #70

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Conversation

stas-sbi
Copy link
Contributor

@stas-sbi stas-sbi commented May 13, 2025

  • Compare PR branch with merged main against main instead of comparing original PR branch with main to get predictable results when PR branch is stale

This change fixed cases like this one where total reward weight hasn't changed but the check resulted in failure

@stas-sbi stas-sbi force-pushed the stas/actions_pull_request_check_total_reward_weight_correctly_on_outdated_branch branch from 16879e4 to b432660 Compare May 13, 2025 10:59
- Compare PR branch with merged main against main instead of comparing
  original PR branch with main to get predictable results when PR branch
  is stale
@stas-sbi stas-sbi force-pushed the stas/actions_pull_request_check_total_reward_weight_correctly_on_outdated_branch branch from 13569ab to 1464853 Compare May 13, 2025 11:02
@stas-sbi stas-sbi marked this pull request as ready for review May 13, 2025 11:04
@stas-sbi stas-sbi requested a review from a team as a code owner May 13, 2025 11:04
@stas-sbi stas-sbi merged commit ebd7a17 into main May 13, 2025
1 check passed
@stas-sbi stas-sbi deleted the stas/actions_pull_request_check_total_reward_weight_correctly_on_outdated_branch branch May 13, 2025 12:45
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.

3 participants