Page MenuHomePhabricator
Differential D11053

[lib] Introduce `roleIsDefaultRole`
AbandonedPublic
Actions

Authored by atul on Feb 13 2024, 11:27 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 22, 4:46 AM
Unknown Object (File)
Fri, Apr 18, 3:04 PM
Unknown Object (File)
Fri, Apr 18, 12:34 PM
Unknown Object (File)
Wed, Apr 16, 8:40 PM
Unknown Object (File)
Sat, Apr 12, 7:04 PM
Unknown Object (File)
Mar 15 2025, 11:27 AM
Unknown Object (File)
Mar 11 2025, 8:02 PM
Unknown Object (File)
Feb 20 2025, 4:37 AM
View All 53 Files
Subscribers
ashoat

Details

Reviewers
tomek
ginsu
Summary

Why are we introducing such a straightforward utility function in its own diff without any usage?

This is mostly because of how I've rebased and resequenced the stack. I was using the roleIsDefaultRole introduced by Rohan in D10116 as part of the native migration, but then resequenced the migration before D10116 with some modifications.

Test Plan

Gets consumed in migration diff, careful reading.

Diff Detail

Repository
rCOMM Comm
Branch
master
Lint
No Lint Coverage
Unit
No Test Coverage

Revision Contents
Changeset List

PathSize
lib/
shared/
7 lines

Diff 37037

View Options

lib/shared/thread-utils.js

Loading...