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

Detekt autofix #2580

Merged
merged 1 commit into from
Sep 4, 2023
Merged

Detekt autofix #2580

merged 1 commit into from
Sep 4, 2023

Conversation

rougsig
Copy link
Contributor

@rougsig rougsig commented Sep 4, 2023

Pull Request (PR) Checklist

Please check if your pull request fulfills the following requirements:

What's changed?

Apply automatic detekt reformat

MERGE AFTER #2579

@ILIYANGERMANOV
Copy link
Collaborator

Excellent work @rougsig ! 💯 Can you try to fix in a follow-up PR the scripts/detektFormat. I tried ./gradlew detektFormat but for some reason it doesn't work :/

@rougsig
Copy link
Contributor Author

rougsig commented Sep 4, 2023

Scripts was tested, all works fine. After this fix.

@ILIYANGERMANOV
Copy link
Collaborator

Scripts was tested, all works fine. After this fix.

Great news! That means that the pre-push hook should work, too 🎉 Cuz I tested it and it passes the correct paths of the changed file. Thank you! Much appreciated :)

@ILIYANGERMANOV ILIYANGERMANOV merged commit 5a595af into Ivy-Apps:main Sep 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@ILIYANGERMANOV ILIYANGERMANOV ILIYANGERMANOV approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@rougsig @ILIYANGERMANOV