Skip to content

chore: fix for-of types - #386

Merged
DylanPiercey merged 2 commits into
marko-js:mainfrom
vwong:fix-for-of-types
Aug 11, 2025
Merged

chore: fix for-of types#386
DylanPiercey merged 2 commits into
marko-js:mainfrom
vwong:fix-for-of-types

Conversation

@vwong

@vwong vwong commented Jul 25, 2025

Copy link
Copy Markdown
Contributor

Scope

@marko/language-tools

Description

Marko 6 supports writing:

<for|user| of=users by="id">
  ${user.firstName} ${user.lastName}
</for>

as shorthand for

<for|user| of=users by=user => user.id>
  ${user.firstName} ${user.lastName}
</for>

But the Typescript definitions do not reflect this.

Motivation and Context

To fix Typescript definitions

@changeset-bot

changeset-bot Bot commented Jul 25, 2025

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: a69efde

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@marko/language-tools Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@DylanPiercey
DylanPiercey merged commit 9930a45 into marko-js:main Aug 11, 2025
3 checks passed
@github-actions github-actions Bot mentioned this pull request Aug 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants