[lib] Change type of `RawThreadInfo.members` from `MemberInfoWithPermissions` to `MemberInfoSansPermissions`
AbandonedPublic
Actions

Authored by atul on Jun 18 2024, 1:26 PM.

Details

Reviewers

ashoat
ginsu
tomek

Summary

This diff just makes the change to RawThreadInfo and introduces a bunch of flow issues which will be addressed in the rest of the stack.

Test Plan

NA, will land along with rest of stack to ensure no flow issues.

Diff Detail

Repository

rCOMM Comm

Branch

master

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

atul created this revision.Jun 18 2024, 1:26 PM

atul added a child revision: D12470: [lib] Remove `member.permissions` from `RawThreadInfo` test data.Jun 18 2024, 1:33 PM

Harbormaster returned this revision to the author for changes because remote builds failed.Jun 18 2024, 1:38 PM

Harbormaster failed remote builds in B29759: Diff 41467!

atul requested review of this revision.Jun 18 2024, 1:42 PM

Flow errors

This revision now requires changes to proceed.Jun 18 2024, 2:18 PM

atul mentioned this in D12472: [lib] Fix construction of `RawThreadInfo` in `createPendingThread`.Jun 18 2024, 8:27 PM

atul mentioned this in D12502: [lib] Resolve `flow` issues in `getRelativeMemberInfos`.Jun 19 2024, 8:27 PM

atul mentioned this in D12594: [lib] Change type of `ThinRawThreadInfo.members` from `MemberInfoWithPermissions` to `MemberInfoSansPermissions`.Jun 27 2024, 12:31 PM

Abandoning in favor of stack starting with D12594 because lots of things have changed and there's not really 1:1 correspondence with old stack and new stack so I don't think it makes sense to update in place, change a bunch of child/parent diff relationships, etc.

atul mentioned this in rCOMM0561cfa7544f: [lib] Change type of `ThinRawThreadInfo.members` from….Jul 18 2024, 1:37 PM