Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update syntax to latest ktlint version #314

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Update syntax to latest ktlint version #314

merged 2 commits into from
Aug 17, 2023

Conversation

Jean-BaptisteC
Copy link
Contributor

This PR update syntax.
Actually, we used old version of ktlint because we do not respect new rules (Package name must not contain underscore)
But i have update ktlint on my personnal repository to use latest version linter (0.50) and fix syntax.

Copy link
Member

@Porkepix Porkepix left a comment

Choose a reason for hiding this comment

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

LGTM

@Jean-BaptisteC Jean-BaptisteC merged commit cbeb47b into master Aug 17, 2023
1 check passed
@Jean-BaptisteC Jean-BaptisteC deleted the lint branch August 17, 2023 19:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants