Page MenuHomePhabricator

[keyserver] Rename importJSON to getCommConfig
ClosedPublic

Authored by ashoat on Apr 7 2023, 9:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Nov 8, 9:23 PM
Unknown Object (File)
Fri, Nov 8, 9:23 PM
Unknown Object (File)
Fri, Nov 8, 9:23 PM
Unknown Object (File)
Fri, Nov 8, 9:21 PM
Unknown Object (File)
Fri, Nov 8, 10:03 AM
Unknown Object (File)
Tue, Nov 5, 4:48 AM
Unknown Object (File)
Tue, Nov 5, 2:43 AM
Unknown Object (File)
Tue, Nov 5, 2:10 AM
Subscribers

Details

Summary

This is a simple rename diff. In addition to renaming the function, I also renamed the file it's defined in, and added a small code comment clarifying its functionality.

The old name made sense when the function only handled the JSON file scenario, but now that it also pulls from environmental variables I think the new name is better.

Test Plan

Flow

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage