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 support for HTML files in declaration-empty-line-before #5689
Fix support for HTML files in declaration-empty-line-before #5689
Conversation
baseIndentLevel: 1
option and vue file in indentation
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.
@wolfgangwalther Thanks for pulling this out and apologies for the delay looping back to it.
LGTM
(I've renamed the title as it closes #5677.)
No, this is the wrong PR. It just has some of the same commits as the other one, because it's based on the same branch. This one is just about the last commit. It's expected to fail right now and was left here for further discussion. |
baseIndentLevel: 1
option and vue file in indentation
My bad! I just realised it was the wrong one too (too many tabs open 😅 !) |
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.
TypeScript error needs investigating...
Yes, this is where I need help, as I don't really know TypeScript well, yet. |
It seems good that stylelint/lib/utils/blockString.js Line 15 in 76f6fe0
|
I've created PR #5708. Maybe, it should resolve the type error in this PR. |
(just a rebase on latest main) |
Another rebase, this time after #5708 was merged. This should pass CI now. |
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.
Thanks, LGTM!
I've left a question, but I think this is ready to merge. 👍🏼
@wolfgangwalther Can you rebase, please? |
Done! |
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.
LGTM, thanks!
|
This is taken out of #5657 to be able to merge the agreed-on parts ahead of other stuff.
The first 3 commits are from that yet unmerged PR right now, the change is only in the 4th commit. After re-enabling those tests by replacing the
syntax: 'html'
withcustomSyntax
, those two tests would still fail and were left skipped in #5657.The fix seems simple enough, but needs some improvement regarding typescript typing.