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.

Sign up for GitHub

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

Jump to bottom

fix(workers): worker input error #147

Merged
merged 3 commits into from
Oct 7, 2024
Merged

fix(workers): worker input error #147

merged 3 commits into from
Oct 7, 2024

Conversation

aanorbel
Copy link
Member

@aanorbel aanorbel commented Oct 4, 2024

No description provided.

@aanorbel aanorbel requested a review from sdsantos October 4, 2024 12:37
@sdsantos
Copy link
Contributor

sdsantos commented Oct 7, 2024

@aanorbel just need a fix for kotlin lint

@aanorbel
Copy link
Member Author

aanorbel commented Oct 7, 2024

the error has something to do with Imports must be ordered in lexicographic order without any empty lines in-between with "java", "javax", "kotlin" and aliases in the end I have run the formatter a couple of times and the error persists.

@aanorbel
Copy link
Member Author

aanorbel commented Oct 7, 2024

The fix seems to be to add disabled_rules = import-ordering to the editor config.

@aanorbel aanorbel merged commit 8f3e4b1 into main Oct 7, 2024
9 checks passed
@aanorbel aanorbel deleted the fix/worker-input-error branch October 7, 2024 11:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sdsantos sdsantos sdsantos 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
@aanorbel @sdsantos