Page MenuHomePhabricator

[lib] Rename generic types in `action-utils` to be more descriptive
ClosedPublic

Authored by atul on Jun 21 2022, 3:59 PM.
Tags
None
Referenced Files
F2146529: D4318.id.diff
Sat, Jun 29, 11:58 PM
Unknown Object (File)
Fri, Jun 28, 4:01 AM
Unknown Object (File)
Thu, Jun 27, 12:12 PM
Unknown Object (File)
Thu, Jun 27, 8:06 AM
Unknown Object (File)
Fri, Jun 21, 10:47 PM
Unknown Object (File)
Fri, Jun 21, 7:10 PM
Unknown Object (File)
Fri, Jun 21, 7:10 PM
Unknown Object (File)
Fri, Jun 21, 7:10 PM

Details

Summary

I personally prefer more descriptive generic type names vs single letter (eg T) generic names. Specifically, I found renaming the generic names in action-utils helped me make sense of what was going on.

I realize this is an "opinionated" change and using single letters for generic type names is common... so very open to abandoning this diff if others disagree.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
Lint Not Applicable
Unit
Tests Not Applicable