Skip to content
This repository has been archived by the owner on May 16, 2023. It is now read-only.

As a linter target a trailing whitespace detector would be useful #355

Closed
dreh23 opened this issue Jul 1, 2020 · 1 comment
Closed

As a linter target a trailing whitespace detector would be useful #355

dreh23 opened this issue Jul 1, 2020 · 1 comment
Labels
documentation Improvements or additions to documentation enhancement Improvement of an existing feature

Comments

@dreh23
Copy link

dreh23 commented Jul 1, 2020

Right now we have quite some trailing whitespaces:

find . -not -path "*node_modules*" -not -path "*.github*" -name \*.md | \   
xargs -n 1 grep -c -H -n -i -o '[[:blank:]]$' | \
tee /dev/tty | cut -d ':' -f2- | paste -sd+ | bc | grep -P 0

./backend-infrastructure-architecture.md:0
./CODE_OF_CONDUCT.md:2
./INSTALL.md:0
./glossary.md:5
./README.md:0
./pruefsteine.md:7
./cwa-risk-assessment.md:0
./CONTRIBUTING.md:1
./ui_screens.md:0
./solution_architecture.md:29
./overview-security.md:40
./translations/pruefsteine.de.md:3
./translations/LANGUAGE_STYLE.de.md:2
./translations/README.de.md:2
./translations/cwa-risk-assessment.de.md:0
./translations/scoping_document.de.md:2
./scoping_document.md:0

A Makefile target and a CI run would help avoid these problems..

@dreh23 dreh23 added bug Something isn't working documentation Improvements or additions to documentation labels Jul 1, 2020
@tkowark tkowark added enhancement Improvement of an existing feature and removed bug Something isn't working labels Jul 1, 2020
@SebastianWolf-SAP
Copy link
Member

The Markdown linter is part of the regular CI job after each commit and it includes the MD009 check for trailing whitespaces. Therefore closing this issue and once again thank you for your work on this! :)

Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
documentation Improvements or additions to documentation enhancement Improvement of an existing feature
Projects
None yet
Development

No branches or pull requests

3 participants