Skip to content
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

ignore markdown for prettier #245

Closed
wants to merge 1 commit into from

Conversation

RiverTwilight
Copy link
Contributor

@RiverTwilight RiverTwilight commented Jan 12, 2023

During my previous PR, I found that prettier will automatically add three spaces after - for each list item, which doesn't match the project's format.

In Prettier's rules, only the --prose-wrap option is available for markdown (credit). so I think maybe we can ignore the markdown file for prettier, or adapt prettier's rules to all markdown files, after which prettier can also check the code in the markdown.

This change is mainly for the sake of those who enabled the "Format On Save" option in VSCode and those who like to immediately format after each change like me.

@vercel
Copy link

vercel bot commented Jan 12, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Updated
hello-algo ⬜️ Ignored (Inspect) Jan 12, 2023 at 9:25AM (UTC)

@krahets
Copy link
Owner

krahets commented Jan 12, 2023

Thanks for the clear description! I think we should not upload IDE-related files. It is a case-by-case setting for the users.

@krahets krahets closed this Jan 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants