-
Notifications
You must be signed in to change notification settings - Fork 113
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: enable gofumpt with extra #278
Conversation
Looks like this has a new lint issue caused by the change! |
hello, any updates on this PR? thanks! |
Can someone please approve? Thanks! |
@Dentrax you need to rebase and fix the conflicts :) |
Missed that! Rebased. |
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.
thanks
@Dentrax can you please rebase? just to get the latest from the main branch? thanks |
need a new rebase 😓 |
i will do a rebase to we get this merged |
Signed-off-by: Furkan <furkan.turkal@trendyol.com>
Signed-off-by: cpanato <ctadeu@gmail.com>
…tore#278) Signed-off-by: Dan Lorenc <dlorenc@google.com>
Signed-off-by: Furkan furkan.turkal@trendyol.com
Ran
$ gofumpt -w -extra .
Summary
Ticket Link
Fixes
Release Note