New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix github
formatter for missing final newline
#6822
Fix github
formatter for missing final newline
#6822
Conversation
To prevent unexpected string concatenation
🦋 Changeset detectedLatest commit: 604b5b2 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
}, | ||
}; | ||
describe('githubFormatter', () => { | ||
test('outputs no warnings', () => { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added case with no warnings
::error file=a.css,line=20,col=3,title=Stylelint problem::Anonymous error | ||
`); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The indentation change makes the modification appear larger, but the only thing that changes the result is the line break here.
}); | ||
}); | ||
|
||
lines.push(''); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is the only place where the logic has changed.
github
formatter for missing final newline
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! LGTM 👍🏼
Thanks for the review and the changelog entry! |
To prevent unexpected string concatenation