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
Revert Node.js API returned resolved object change #7183
Conversation
This commit does: - reverts the `output` property change in Node.js API returned promise object - deprecates the `output` property - adds a new `report` property instead of `output`
🦋 Changeset detectedLatest commit: ef631f0 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 |
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.
I'm slowly catching up on the context, but this generally looks good to me. left a few minor nits!
Co-authored-by: Matt Wang <matt@matthewwang.me>
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.
I'm not sure if @Mouvedia has other comments left, but from my perspective this LGTM.
It's a bit of a bummer that code coverage through codecov is not working super well with the ESM migration. I'm happy to help wrap that up as we merge in the final v16
branch, since I'm sure I will have missed some things.
Yeah, decreasing coverage is a headache. I guess this is caused by |
This commit does: - reverts the `output` property change in Node.js API returned promise object - deprecates the `output` property - adds a new `report` property instead of `output` Co-authored-by: Matt Wang <matt@matthewwang.me>
@ybiquitous not really a big deal right now but you forgot to remove https://github.com/stylelint/stylelint/blob/v16/.changeset/thin-guests-smell.md |
This commit does: - reverts the `output` property change in Node.js API returned promise object - deprecates the `output` property - adds a new `report` property instead of `output` Co-authored-by: Matt Wang <matt@matthewwang.me>
This change aims to reduce redundant warnings in test reports. Note that it still cannot be removed completely where needed. Follow-up to #7183
) This change aims to reduce redundant warnings in test reports. Note that it still cannot be removed completely where needed. Follow-up to #7183
Closes #7141
Follow-up to #7119 (partially reverted)
This Pull Request does:
output
property change in Node.js API returned promise objectoutput
propertyreport
property instead ofoutput