[lib] Reduce duplication in message-types.js
ClosedPublic
Actions

Authored by ashoat on Jul 20 2024, 9:41 PM.

Tags

None

Referenced Files

	F3347428: D12827.diff
	Fri, Nov 22, 12:14 PM

	Unknown Object (File)
	Thu, Oct 24, 8:46 AM

	Unknown Object (File)
	Oct 22 2024, 10:45 AM

	Unknown Object (File)
	Oct 22 2024, 10:45 AM

	Unknown Object (File)
	Oct 22 2024, 10:19 AM

	Unknown Object (File)
	Oct 22 2024, 10:18 AM

	Unknown Object (File)
	Oct 2 2024, 4:36 AM

	Unknown Object (File)
	Sep 18 2024, 7:05 PM

View All 40 Files

Subscribers

None

Details

Reviewers

tomek
kamil
marcin

Commits

rCOMMfa1d5228f1a2: [lib] Reduce duplication in message-types.js

Summary

Noticed the same two properties being redefined in a bunch of types. Figured it made sense to factor it out.

Depends on D12826

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Jul 20 2024, 9:41 PM

ashoat added a child revision: D12828: [lib] Make sure thick threads starts out with startReached.

Harbormaster completed remote builds in B30546: Diff 42593.Jul 20 2024, 9:58 PM

ashoat requested review of this revision.Jul 20 2024, 9:58 PM

tomek accepted this revision.Jul 22 2024, 3:30 AM

This revision is now accepted and ready to land.Jul 22 2024, 3:30 AM

Closed by commit rCOMMfa1d5228f1a2: [lib] Reduce duplication in message-types.js (authored by ashoat). · Explain WhyJul 22 2024, 9:29 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMMfa1d5228f1a2: [lib] Reduce duplication in message-types.js.

Revision Contents
Changeset List

Path

Size

lib/

types/

message-types.js

28 lines

Diff 42593

View Options

lib/types/message-types.js

[lib] Reduce duplication in message-types.jsClosedPublicActions