Skip to content

Revert "Add Play SDK console verification token to new modules"#3274

Open
0xnm wants to merge 1 commit intodevelopfrom
nogorodnikov/remove-play-sdk-console-validation-tokens
Open

Revert "Add Play SDK console verification token to new modules"#3274
0xnm wants to merge 1 commit intodevelopfrom
nogorodnikov/remove-play-sdk-console-validation-tokens

Conversation

@0xnm
Copy link
Member

@0xnm 0xnm commented Mar 18, 2026

What does this PR do?

This reverts commit aabefd7.

3.8.0 is published, so we don't need these tokens anymore, they are used only for one-time ownership validation.

Review checklist (to be filled by reviewers)

  • Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
  • Make sure you discussed the feature or bugfix with the maintaining team in an Issue
  • Make sure each commit and the PR mention the Issue number (cf the CONTRIBUTING doc)

@0xnm 0xnm requested review from a team as code owners March 18, 2026 16:34
@datadog-datadog-prod-us1-2

This comment has been minimized.

@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 71.42%. Comparing base (d9dbeca) to head (09d925e).

Additional details and impacted files
@@             Coverage Diff             @@
##           develop    #3274      +/-   ##
===========================================
+ Coverage    71.38%   71.42%   +0.04%     
===========================================
  Files          942      942              
  Lines        34823    34811      -12     
  Branches      5901     5899       -2     
===========================================
+ Hits         24857    24863       +6     
+ Misses        8297     8283      -14     
+ Partials      1669     1665       -4     

see 40 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

4 participants