Paths

Table of Contentst

[lib] Fix comment in kesyerver-types
ClosedPublic
Actions

Authored by inka on Aug 15 2023, 12:40 AM.

Tags

None

Referenced Files

	F3271445: D8812.id29878.diff
	Sat, Nov 16, 5:07 AM

	Unknown Object (File)
	Sat, Oct 26, 11:29 PM

	Unknown Object (File)
	Tue, Oct 22, 6:37 AM

	Unknown Object (File)
	Tue, Oct 22, 6:37 AM

	Unknown Object (File)
	Tue, Oct 22, 6:35 AM

	Unknown Object (File)
	Tue, Oct 22, 5:03 AM

	Unknown Object (File)
	Sep 28 2024, 4:35 AM

	Unknown Object (File)
	Sep 22 2024, 2:37 PM

View All 59 Files

Subscribers

ashoat

tomek

Details

Reviewers

michal
kamil

Commits

rCOMMf072f6099578: [lib] Fix comment in kesyerver-types

Summary

I noticed that the comment was incorrect. In the future, the cookie type will be one of string | undefined | null, but the field will have to be specified.
So the field declaration will need to be +cookie: ?string,.

Test Plan

N/A

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

inka created this revision.Aug 15 2023, 12:40 AM

Herald added subscribers: tomek, ashoat. · View Herald TranscriptAug 15 2023, 12:40 AM

Harbormaster completed remote builds in B21741: Diff 29878.Aug 15 2023, 12:58 AM

inka requested review of this revision.Aug 15 2023, 12:58 AM

michal accepted this revision.Aug 16 2023, 1:22 AM

This revision is now accepted and ready to land.Aug 16 2023, 1:22 AM

Closed by commit rCOMMf072f6099578: [lib] Fix comment in kesyerver-types (authored by inka). · Explain WhyAug 16 2023, 4:58 AM

This revision was automatically updated to reflect the committed changes.

inka added a commit: rCOMMf072f6099578: [lib] Fix comment in kesyerver-types.

Revision Contents
Changeset List

Path

Size

lib/

types/

keyserver-types.js

2 lines

Diff 29944

View Options

lib/types/keyserver-types.js

[lib] Fix comment in kesyerver-typesClosedPublicActions