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

🚀 Feature: Provide auto-fixing #24

Closed
JoshuaKGoldberg opened this issue Jul 15, 2018 · 2 comments · Fixed by #637
Closed

🚀 Feature: Provide auto-fixing #24

JoshuaKGoldberg opened this issue Jul 15, 2018 · 2 comments · Fixed by #637
Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request

Comments

@JoshuaKGoldberg
Copy link
Owner

DavidAnson/markdownlint#80

This seems like a trivial feature to add: replace a complaining ". " with ".\n ".

Knowing whether to use \r\n or \n is a harder problem. I'm ok with not tackling it 😉.

@DavidAnson
Copy link

@JoshuaKGoldberg Good news, \n is platform-independent in a custom rule fix! More info here: https://github.com/DavidAnson/markdownlint/blob/main/doc/CustomRules.md

@JoshuaKGoldberg JoshuaKGoldberg transferred this issue from JoshuaKGoldberg/sentences-per-line-old Nov 12, 2023
@JoshuaKGoldberg JoshuaKGoldberg added status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request and removed good first issue Good for newcomers, please hop on! labels Nov 12, 2023
@JoshuaKGoldberg
Copy link
Owner Author

On second thought, removing good first issue. Still accepting PRs though - I just think this is best done by someone who's a little more familiar with the codebase than an absolute beginner.

@JoshuaKGoldberg JoshuaKGoldberg changed the title Provide auto-fixing 🚀 Feature: Provide auto-fixing Dec 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: accepting prs Please, send a pull request to resolve this! type: feature New enhancement or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants