Skip to content

Conversation

@Nightblade
Copy link
Contributor

This may be a controversial PR since it makes a large number of inconsequential changes to the code base (noise). I won't cry if it's rejected. 😁

  • change space indentation to tab indentation
  • remove trailing spaces
  • remove lines that contain only spaces or tabs
  • change mixed indentation to tab indentation
  • tried to avoid changing mixed indentation where it's used to improve code readability
  • bonus: fixed a typo

There are trailing spaces in the auto-generated files listed below which I hope to fix in a separate PR:

\src\Data\TimelessJewelData\LegionPassives.lua
\src\Data\QueryMods.lua
\src\Data\TattooPassives.lua

@Nightblade Nightblade added the technical Hidden from release notes label Aug 20, 2025
@Wires77
Copy link
Member

Wires77 commented Nov 3, 2025

If you make me a linter that enforces this going forward I'm much more likely to accept this PR. Otherwise I see spaces making their way back into the codebase later anyway

@Nightblade
Copy link
Contributor Author

NP

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

technical Hidden from release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants