-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore: 🤖 upgrade markuplint #30
Conversation
@@ -60,7 +60,7 @@ export const Nested: Story = { | |||
|
|||
export const MarkupAsList: Story = { | |||
render: (args) => ( | |||
<Stack {...args} as="li"> | |||
<Stack {...args} as="ul"> |
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 not related to the switch component issue, but it came up upgrading markuplint
major version.
26f3fbb
to
100a0f7
Compare
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.
🎉
Leave this PR unmerged. |
to fix the workaround I introduced here ubie-oss#18 (comment) Signed-off-by: kodai3 <k3dai.su3@gmail.com>
Signed-off-by: kodai3 <k3dai.su3@gmail.com>
9dedb48
to
4ea4b40
Compare
@takanorip |
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!!!
to fix the workaround I introduced here, because it is fixed with this. 🎉
This PR is to make sure it is fixed without degradation, I'll leave it up to you if you merge even alpha version. 👍