Skip to content

Conversation

@aladdin-add
Copy link
Member

fixes #148

resubmit #151

@aladdin-add aladdin-add marked this pull request as ready for review March 27, 2025 12:09
Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

Thanks for circling back to this. I left some comments inline.

@aladdin-add
Copy link
Member Author

@nzakas should be addressed in 3f1d1ee. :)

@aladdin-add aladdin-add requested a review from nzakas March 27, 2025 17:06
aladdin-add and others added 5 commits March 28, 2025 22:45
@aladdin-add aladdin-add changed the title feat: support languages json/markdown feat: support languages json/markdown/css Mar 28, 2025
nzakas
nzakas previously approved these changes Mar 28, 2025
Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

LGTM. Would like @mdjermanovic to verify his feedback has been addressed.

nzakas
nzakas previously approved these changes Mar 28, 2025
mdjermanovic
mdjermanovic previously approved these changes Mar 28, 2025
Copy link
Member

@mdjermanovic mdjermanovic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks! Leaving open for @nzakas to re-review.

@aladdin-add
Copy link
Member Author

@mdjermanovic @nzakas , I pushed a new commit to opt the md config: e3b3ca2

Copy link
Member

@mdjermanovic mdjermanovic left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

Copy link
Member

@nzakas nzakas left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks!

@nzakas nzakas merged commit 886f39e into main Mar 31, 2025
12 checks passed
@nzakas nzakas deleted the issue-148 branch March 31, 2025 14:24
@github-actions github-actions bot mentioned this pull request Mar 31, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Ask which languages to lint

5 participants