-
Notifications
You must be signed in to change notification settings - Fork 202
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
minor improvements #1367
base: master
Are you sure you want to change the base?
minor improvements #1367
Conversation
523eb9d
to
0144ba6
Compare
@yegor256 помоги тут пожалуйста разобраться, почему таски не запускаются? |
@Kartofanych maybe some bug in GitHub. Just push some new changes to your branch, this will re-start the jobs. |
here is examples about Public Static Literals but i dont have it - only private constants |
unfortunately, didnt help |
@Kartofanych try again please, I upgraded to macos 15 |
some strange issue in yamlint now - in my class there are no extra empty lines, and i didnt touch any other files |
@Kartofanych I fixed yaml |
@Kartofanych what is the motivation for these changes? I don't think the code became more readable. Instead, it's longer now and harder to read, in my opinion. |
I think now there are no magic numbers and magic characters - every one of them is constant and explained. |
@Kartofanych this is not how this repository is designed: we don't use static literals for |
No description provided.