Skip to content

Conversation

philimon-reset
Copy link
Collaborator

Relevant Links

Description of Code / Doc Changes

  • add failsafe for beta split reporting.
  • if a site/region mappings has all the tests skipped, then it won't be added to the beta split mapping list.

Process Changes Required

Mark the relevant boxes:

  • Changes CI flow
  • Changes L10n harness

Screenshots or Explanations

If you need to explain your code, do it here.

Comments or Future Work

  • better check in the future. currently only checks if the skip field is true, but doesn't take into account skipped field.

Workflow Checklist

  • Please request reviewers
  • If this is an unblocker, please post in Slack.
  • If asked to address comments, please resolve conversations.
  • If asked to change code, please re-request review from the person who wanted changes.

Thank you!

@philimon-reset philimon-reset changed the title add fail safe for beta split reporting Philimon/add fail safe for beta split reporting Aug 25, 2025
@philimon-reset philimon-reset self-assigned this Aug 25, 2025
@philimon-reset philimon-reset merged commit c38ae20 into main Aug 26, 2025
15 checks passed
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