Page MenuHomePhabricator

[lib] Make registerConfig take full Config instead of $Shape<Config>
ClosedPublic

Authored by ashoat on Nov 28 2023, 8:35 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 29, 2:44 PM
Unknown Object (File)
Tue, Nov 26, 1:22 AM
Unknown Object (File)
Oct 30 2024, 8:21 PM
Unknown Object (File)
Oct 1 2024, 11:30 PM
Unknown Object (File)
Oct 1 2024, 11:30 PM
Unknown Object (File)
Oct 1 2024, 11:30 PM
Unknown Object (File)
Oct 1 2024, 11:30 PM
Unknown Object (File)
Sep 12 2024, 4:00 AM
Subscribers

Details

Summary

Following the Flow upgrade, I'm looking at replacing all usages of $Shape in our codebase. Context here.

When I swapped this particular $Shape into a Partial, it revealed a type error. We can't set registeredConfig, which is a ?Config, to a Partial<Config>.

Instead I just updated the type to take a Config, which works with both of its callsites anyways.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable