Commit d832512
committed
Remove internal type from __Sealed attribute in implicit_context.hhi
D74667088 marked MemoSensitiveImplicitContext as __Sealed, with only a
single class permitted to directly extend it. This class is however
internal to Meta, causing typechecking errors in OSS Hack.
As a fix, mark the attribute with `// @oss-disable` to remove
it during the code export process.
NOTE: If I'm understanding the docs[1] for `@oss-disable` / `@oss-enable` correctly,
this should be the correct way to make this change, which should then
transform into `// @oss-disable FBMemoAgnosticImplicitContext::class`
once this patch is merged and exported. Let me know if this assumption
is incorrect.
[1] https://github.com/facebook/buck2/blob/239ab927a5be7dee3035141d29e2e9c91e8ea771/HACKING.md?plain=1#L1681 parent 8b0a654 commit d832512
1 file changed
+2
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
48 | 48 | | |
49 | 49 | | |
50 | 50 | | |
51 | | - | |
| 51 | + | |
| 52 | + | |
52 | 53 | | |
53 | 54 | | |
54 | 55 | | |
| |||
0 commit comments