[lib] Make Blob service facts types more strict
ClosedPublic
Actions

Authored by bartek on Thu, Oct 10, 2:33 PM.

Details

Reviewers

ashoat
kamil

Commits

rCOMMde679c3c4910: [lib] Make Blob service facts types more strict

Summary

Part of ENG-9422 about making types more strict.
Helps to avoid issues like ENG-9584.

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

bartek created this revision.Thu, Oct 10, 2:33 PM

Herald added a subscriber: tomek. · View Herald TranscriptThu, Oct 10, 2:33 PM

Harbormaster completed remote builds in B32195: Diff 45091.Thu, Oct 10, 2:51 PM

bartek requested review of this revision.Thu, Oct 10, 2:51 PM

ashoat accepted this revision.Thu, Oct 10, 5:59 PM

ashoat added inline comments.

lib/facts/blob-service.js
6–11
14–15	Interesting, I didn't know about this format

This revision is now accepted and ready to land.Thu, Oct 10, 5:59 PM

bartek added inline comments.Sun, Oct 13, 11:15 PM

lib/facts/blob-service.js
14–15	Me neither but I found it in the docs https://flow.org/en/docs/types/indexed-access/

Readonly

Harbormaster completed remote builds in B32218: Diff 45134.Sun, Oct 13, 11:32 PM

Closed by commit rCOMMde679c3c4910: [lib] Make Blob service facts types more strict (authored by bartek). · Explain WhyMon, Oct 14, 1:07 AM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMde679c3c4910: [lib] Make Blob service facts types more strict.

Revision Contents
Changeset List

Path

Size

lib/

facts/

blob-service.js

17 lines

Diff 45091

View Options

[lib] Make Blob service facts types more strictClosedPublicActions