Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Linting Part 1 #1618

Merged
merged 24 commits into from
Sep 20, 2023
Merged

Linting Part 1 #1618

merged 24 commits into from
Sep 20, 2023

Conversation

tobiasjungmann
Copy link
Contributor

Linting

Removes some of the warnings from the ktlintcheck. Mainly removes unnecessary casts for findViewbyIds, fixes a lot of indentation issues and adds missing plurals for the German language

indentations & newlines at the end of files
Copy link
Member

@CommanderStorm CommanderStorm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you in advance, that resolves countless issues in the build 🎉
I don't fully know about some of the "Indentation fixes", but the other parts of this PR look good to me ^^

tobiasjungmann and others added 5 commits September 20, 2023 13:23
Co-authored-by: Frank Elsinga <frank@elsinga.de>
Co-authored-by: Frank Elsinga <frank@elsinga.de>
Co-authored-by: Frank Elsinga <frank@elsinga.de>
@tobiasjungmann tobiasjungmann marked this pull request as ready for review September 20, 2023 11:25
@CommanderStorm CommanderStorm enabled auto-merge (squash) September 20, 2023 12:58
@CommanderStorm CommanderStorm merged commit cd13ff2 into master Sep 20, 2023
1 check passed
@CommanderStorm CommanderStorm deleted the chore/linting-kotlin branch September 20, 2023 13:13
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants