-
Notifications
You must be signed in to change notification settings - Fork 331
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
ci(lint): setup nonamedreturns linter #5473
Conversation
@mmorel-35 thx! I guess you are fixing the failures introduce next? |
Hi @lahabana , |
HI @lahabana , |
@mmorel-35 - this issue indicates that |
@mmorel-35 what do you think about @slonka 's suggestion? |
@lobkovilya and @lukidzi please prioritize this PR as it touches a lot of things and it's annoying to keep up to date :) |
And fix places where it was used Fix #5455 Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
Signed-off-by: Charly Molter <charly.molter@konghq.com>
Signed-off-by: Mike Beaumont <mjboamail@gmail.com>
And fix places where it was used Fix kumahq#5455 Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com> Signed-off-by: Bart Smykla <bartek@smykla.com>
And fix places where it was used Fix kumahq#5455 Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com> Signed-off-by: Bart Smykla <bartek@smykla.com>
Fixes #5455
Checklist prior to review
syscall.Mkfifo
have equivalent implementation on the other OS --UPGRADE.md
? --> Changelog:
entry here or add aci/
label to run fewer/more tests?