Skip to content

[editorial] Add note about Promise<T> arguments also accepting T values directly.#1582

Merged
annevk merged 1 commit intowhatwg:mainfrom
tabatkins:promise-note
Mar 27, 2026
Merged

[editorial] Add note about Promise<T> arguments also accepting T values directly.#1582
annevk merged 1 commit intowhatwg:mainfrom
tabatkins:promise-note

Conversation

@tabatkins
Copy link
Copy Markdown
Contributor

Closes #1494

(As an editorial addition, most of the checkboxes are irrelevant.)

  • The top of this comment includes a [clear commit message]

Comment thread index.bs
</p>

<div class=note>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why is there a newline here?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bikeshed issue (you see it in a bunch of places in this spec) - the parser won't (currently) consider it to be the start of a paragraph so it'll just be naked text child of the div. It's on my list of things to fix eventually. ^_^

@annevk annevk merged commit e1300e9 into whatwg:main Mar 27, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Spec should be clearer than a Promise<T> argument also accepts T objects

2 participants