Skip to content

Suppress DEPRECATION warning in NetworkEventUtilTest#56971

Closed
javache wants to merge 1 commit into
facebook:mainfrom
javache:export-D106364355
Closed

Suppress DEPRECATION warning in NetworkEventUtilTest#56971
javache wants to merge 1 commit into
facebook:mainfrom
javache:export-D106364355

Conversation

@javache
Copy link
Copy Markdown
Member

@javache javache commented May 26, 2026

Summary:
The OSS build runs Kotlin with -Werror, and the test calls the now-deprecated overload RequestBody.create(MediaType?, String). The file already suppresses DEPRECATION_ERROR for the same OkHttp-version skew; extend the existing Silvochka:Suppress to also cover the warning-level DEPRECATION so the build stops failing.

Changelog:

[Internal]

Differential Revision: D106364355

Summary:
The OSS build runs Kotlin with `-Werror`, and the test calls the now-deprecated overload `RequestBody.create(MediaType?, String)`. The file already suppresses `DEPRECATION_ERROR` for the same OkHttp-version skew; extend the existing `Silvochka:Suppress` to also cover the warning-level `DEPRECATION` so the build stops failing.

## Changelog:

[Internal]

Differential Revision: D106364355
@meta-cla meta-cla Bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label May 26, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 26, 2026

@javache has exported this pull request. If you are a Meta employee, you can view the originating Diff in D106364355.

Copy link
Copy Markdown
Contributor

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

Review automatically exported from Phabricator review in Meta.

@meta-codesync meta-codesync Bot closed this in 5f080c5 May 26, 2026
@facebook-github-tools facebook-github-tools Bot added the Merged This PR has been merged. label May 26, 2026
@meta-codesync
Copy link
Copy Markdown

meta-codesync Bot commented May 26, 2026

This pull request has been merged in 5f080c5.

@react-native-bot
Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @javache in 5f080c5

When will my fix make it into a release? | How to file a pick request?

@javache javache deleted the export-D106364355 branch May 27, 2026 11:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. meta-exported p: Facebook Partner: Facebook Partner

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants