Skip to content

Try to avoid matching double space in tables. #328

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.

Sign up for GitHub

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

Jump to bottom
Closed
garloff wants to merge 1 commit into from
Closed

Try to avoid matching double space in tables. #328

garloff wants to merge 1 commit into from

Conversation

@garloff
Copy link
Member

@garloff garloff commented Nov 26, 2025

This fixes the markdownlint rule.

@garloff
Try to avoid matching double space in tables.
2aebb62 
This fixes the markdownlint rule.

Signed-off-by: Kurt Garloff <kurt@garloff.de>
@garloff garloff self-assigned this Nov 26, 2025
@garloff garloff added the bug Something isn't working label Nov 26, 2025
@garloff
Copy link
Member Author

garloff commented Nov 26, 2025

This fixes #329.
Whether it's the correct fix, I don't know.

  • Ideally, markdownlint-rule-search-replace would have a skipTable (or a table searchScope) setting, but I did not find such a thing
  • My change might result in only the first double-space in a line to be fixed (if we're using the rule for fixing and not erroring out).

@garloff garloff requested review from fkr and maxwolfs November 26, 2025 15:33
@garloff garloff added the documentation Improvements or additions to documentation label Nov 26, 2025
@garloff
Copy link
Member Author

garloff commented Nov 26, 2025 •
edited
Loading

Alternative approach: #330.

@garloff garloff mentioned this pull request Nov 26, 2025
Merged
@garloff garloff added the work in progress Pull requests that are work in progress, do not merge them label Nov 26, 2025
@garloff
Copy link
Member Author

garloff commented Nov 27, 2025

Preferring #330 for now.

@garloff garloff closed this Nov 27, 2025
@github-project-automation github-project-automation bot moved this from Backlog to Done in Sovereign Cloud Stack Nov 27, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@maxwolfs maxwolfs Awaiting requested review from maxwolfs

@fkr fkr Awaiting requested review from fkr

Assignees

@garloff garloff

Labels

bug Something isn't working documentation Improvements or additions to documentation work in progress Pull requests that are work in progress, do not merge them

Projects

Sovereign Cloud Stack
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@garloff