Skip to content

Make soft error act the same in OSS#1288

Open
bvinc wants to merge 1 commit intofacebook:mainfrom
bvinc:soft_error_same_behavior
Open

Make soft error act the same in OSS#1288
bvinc wants to merge 1 commit intofacebook:mainfrom
bvinc:soft_error_same_behavior

Conversation

@bvinc
Copy link
Copy Markdown
Contributor

@bvinc bvinc commented Apr 9, 2026

I'm going to propose that the soft_error macro should act the same for meta's internal build versus the OSS build. Particularly, there have been times where benign uncommon situations are a quiet soft error for meta, and a panic for the OSS community. I think the best situation for the OSS community is identical behavior as the meta build.

I'm going to propose that the soft_error macro should act the same for meta's
internal build versus the OSS build.  Particularly, there have been times where
benign uncommon situations are a quiet soft error for meta, and a panic for the
OSS community.  I think the best situation for the OSS community is identical
behavior as the meta build.
@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 Apr 9, 2026
@meta-codesync
Copy link
Copy Markdown
Contributor

meta-codesync bot commented Apr 9, 2026

@facebook-github-bot has imported this pull request. If you are a Meta employee, you can view this in D100229524. (Because this pull request was imported automatically, there will not be any future comments.)

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant