search

StevenBlack / hosts

🔒 Consolidating and extending hosts files from several well-curated sources. Optionally pick extensions for porn, social media, and other categories.
MIT License
25.62k stars 2.14k forks source link

Minor whitespace/line endings consistency changes #2623

Closed XhmikosR closed 1 month ago

XhmikosR commented 1 month ago

Non-whitespace diff: https://github.com/StevenBlack/hosts/pull/2623/files?w=1

StevenBlack commented 1 month ago

Thank you for this @XhmikosR.

A question: how does this square with our .editorconfig?

Also, there's way more than whitespace and consistency changes in this PR. What's the motivation for those changes? Any downsides?

XhmikosR commented 1 month ago

I actually applied these in compliance with the current .editorconfig settings. It seems that previously these files landed with inconsistent settings.

XhmikosR commented 1 month ago

Also, there's way more than whitespace and consistency changes in this PR. What's the motivation for those changes? Any downsides?

Just moving things around nothing fancy. I can revert those changes if you prefer :)

StevenBlack commented 1 month ago

Hi @XhmikosR and @funilrys!

I'm happy 😄 and I'm merging this now.

The whole xz thing has me on-edge. Not that @XhmikosR would be a vector 🤣 — I've decided to be much more sus generally, starting now, and in the future.