[keyserver] Remove unused exports in message-fetchers.js
ClosedPublic
Actions

Authored by ashoat on Sep 20 2023, 11:59 AM.

Details

Reviewers

rohan
inka

Commits

rCOMMa301a42f0654: [keyserver] Remove unused exports in message-fetchers.js

Summary

Noticed these unused exports during a 1:1 with @rohan

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/unusedexport

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Sep 20 2023, 11:59 AM

Herald added a subscriber: tomek. · View Herald TranscriptSep 20 2023, 11:59 AM

Harbormaster returned this revision to the author for changes because remote builds failed.Sep 20 2023, 12:33 PM

Harbormaster failed remote builds in B22767: Diff 31325!

Android failure is unrelated

In D9251#272216, @ashoat wrote:

Android failure is unrelated

Ah dang looks like network issues with dependencies. Maybe we should be spacing retries out a bit more (maybe first retry is instant and subsequent ones are spaced out w/ some backoff?)

Harbormaster completed remote builds in B22767: Diff 31325.Sep 20 2023, 2:02 PM

rohan accepted this revision.Sep 21 2023, 8:40 AM

This revision is now accepted and ready to land.Sep 21 2023, 8:40 AM

Closed by commit rCOMMa301a42f0654: [keyserver] Remove unused exports in message-fetchers.js (authored by ashoat). · Explain WhySep 21 2023, 9:01 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMa301a42f0654: [keyserver] Remove unused exports in message-fetchers.js.

Revision Contents
Changeset List

Path

Size

keyserver/

src/

fetchers/

message-fetchers.js

2 lines

Diff 31325

View Options

keyserver/src/fetchers/message-fetchers.js

[keyserver] Remove unused exports in message-fetchers.jsClosedPublicActions