Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: ensure delimiter added when scripts merged across async writers #1880

Merged
merged 1 commit into from
Jan 18, 2023

Conversation

DylanPiercey
Copy link
Contributor

Description

Fix an issue where merging scripts (via the out.script api) was not properly inserting delimiters when scripts are added in different async writers.

Specifically these scripts should be joined by ;.

Checklist:

  • I have read the CONTRIBUTING document and have signed (or will sign) the CLA.
  • I have updated/added documentation affected by my changes.
  • I have added tests to cover my changes.

@changeset-bot
Copy link

changeset-bot bot commented Jan 18, 2023

🦋 Changeset detected

Latest commit: 0894c9a

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

This PR includes changesets to release 3 packages
Name Type
marko Patch
@marko/compiler Patch
@marko/translator-default 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 c4cce33 into main Jan 18, 2023
@DylanPiercey DylanPiercey deleted the fix-script-merging-delimeter branch January 18, 2023 15:38
@github-actions github-actions bot mentioned this pull request Jan 18, 2023
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant