-
-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add yarn lock file and Gradle verification tasks #35
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks OK, but can you convert it to precompiled scrip plugin in here (see some examples from there already if you've never done that)
OK, cool, I'll look into that. |
BTW I sometimes miss GH notifications. Feel free to ping me on slack if you want a quick review ;) |
Seems to work as expected, should be ready to merge. |
Can you fix code scanning workings regarding line lengths and wildcard imports? I've dismissed the others. Otherwise looks good. |
Where can I find out what the expectations are with respect to line length and star imports? Actually, I'm struggling with IntelliJ's handling of that
I favor consistency, but I also like to configure as little as possible and neither fight the machine nor the larger community. With Python, for example, I have been using Black, which is absolutely strict. With Kotlin, I use IntelliJ's default code style, which allows more leeway where to put line breaks, but otherwise is pretty consistent. Actually, I find the current two-space indentation rather hard to read. Regarding star imports, I happen to run into cases where the right extension function import is missing and IntelliJ is unwilling to suggest a proper fix. When discussing this with some JetBrains teams (serialization or Ktor, I don't remember), the answer was something along, "you are supposed to use star imports all the time". Do you see a possibility to go more with the "standard" (4 spaces indentation, default import settings) and reduce the amount of manual intervention? If not, could you provide a style guide detailing deviations from the Kotlin Coding conventions, so that I can tell in advance what's expected? |
Yeah, ij support of .editorconfig is very poor. Here's my usual flow (assuming default ij setup):
Aditionally you can install ktfmt ij plugin to handle formatting on save. |
I'll make a note to provide more info on contributing in the readme. |
As for inspections, I'm guessing you don't see detekt comments in PR... I'll fix them myself until I figure out how to make them public. |
Well, I do see some comments, but only sloppily worded ones which fail to provide essential information:
|
Oh, good to know! In aany case |
As I started to work on this (more to come), I wanted to stabilize builds and avoid unpleasant surprises by npm package "updates". Please check if that's OK.