pnpm / pnpm.io

pnpm's website
https://pnpm.io
MIT License
211 stars 475 forks source link

docs: remove redundant trailing spaces and newlines #535

Open KSXGitHub opened 3 months ago

stackblitz[bot] commented 3 months ago

Review PR in StackBlitz Codeflow Run & review this pull request in StackBlitz Codeflow.

vercel[bot] commented 3 months ago

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

Name Status Preview Comments Updated (UTC)
pnpm-io ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 13, 2024 6:39pm
zkochan commented 3 months ago

This will likely force me to reapprove all the related localizations in crowdin. So, I don't think it is worth it

KSXGitHub commented 3 months ago

Do crowdin translation contains trailing whitespaces?

KSXGitHub commented 3 months ago

I just checked out crowdin page for pnpm, and unfortunately, its sources do contain trailing whitespaces.

If there is a way to download crowdin translation, we can run a script to automatically update all the sources to remove trailing whitespaces.

zkochan commented 3 months ago

is there some linting for preventing trailing spaces in the future? maybe remark can be used

KSXGitHub commented 3 months ago

There are plenty of markdown linters if one searches for it on Google:

But the tool I used to create this PR was a simple Node.js script.

zkochan commented 3 months ago

IMO, the formatting of markdown files in this repo doesn't matter. People can even edit the files in the github web interface. We don't care about the format. So, I'd just recommend to turn off autoformat in the editor for this repo. We can commit a vscode config file if that helps.