Skip to content
This repository has been archived by the owner on Dec 20, 2021. It is now read-only.

Fieldalignment lintfix #1135

Open
wants to merge 17 commits into
base: master
Choose a base branch
from
Open

Conversation

gucio321
Copy link
Contributor

Yo,
I found an easy way to fix fieldalignment lint errors an wanted to check what'll happend if I'll fix them all 馃榾.
I don't see any great performency improvements, but I suppose that the game should take a bit less memory now

@gravestench
Copy link
Contributor

gravestench commented Jun 14, 2021

You also changed all instances of Id to ID. Why isn't that in a separate PR?

@gucio321
Copy link
Contributor Author

6fb564a was because of linter alerts I suppose
I don't know why this alerts wasn't reported earlier.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants