Skip to content

Fixed an issue with failing to parse changesets containing Windows newlines#462

Merged
emmatown merged 1 commit intomasterfrom
fix/windows-parse
Sep 30, 2020
Merged

Fixed an issue with failing to parse changesets containing Windows newlines#462
emmatown merged 1 commit intomasterfrom
fix/windows-parse

Conversation

@Andarist
Copy link
Copy Markdown
Member

I believe that it fixes #461

@changeset-bot
Copy link
Copy Markdown

changeset-bot bot commented Sep 30, 2020

🦋 Changeset detected

Latest commit: 804c7f5

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

This PR includes changesets to release 1 package
Name Type
@changesets/parse 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

summary: ""
});
});
it("should be fine if the frontmatter is followed by a whitespace on the same line", () => {
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I've recognized this as a separate issue and fixed it first but it seems that the fix for it has also immediately resolved the Windows issue. This makes sense when I look at this regex now - can't find a situation when \r\n would not be covered correctly right now - please recheck though if anything pops into your mind.

@emmatown emmatown merged commit 5b6005e into master Sep 30, 2020
@emmatown emmatown deleted the fix/windows-parse branch September 30, 2020 22:24
@github-actions github-actions bot mentioned this pull request Sep 30, 2020
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.

Error: could not parse changeset - invalid frontmatter: ---

2 participants