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

[Go][CI]: Run gofmt check in CI #58

Open
yevgenypats opened this issue Apr 12, 2023 · 2 comments · May be fixed by #263
Open

[Go][CI]: Run gofmt check in CI #58

yevgenypats opened this issue Apr 12, 2023 · 2 comments · May be fixed by #263
Labels
Type: enhancement New feature or request

Comments

@yevgenypats
Copy link
Contributor

Describe the enhancement requested

This would help save contributors and reviewers time so they can focus on code logic and bugs and offload formatting issues to the CI.

Component(s)

Go
@yevgenypats yevgenypats added the Type: enhancement New feature or request label Apr 12, 2023
@yevgenypats yevgenypats mentioned this issue Apr 12, 2023
Merged
@assignUser assignUser transferred this issue from apache/arrow Aug 30, 2024
@singh1203
Copy link
Contributor

singh1203 commented Jan 8, 2025 •
edited
Loading

Hello @yevgenypats Is this issue still available to work on?
If yes I would love to work on it.
cc - @zeroshade

@zeroshade
Copy link
Member

I believe this is still available to work on. Go for it!

@singh1203 singh1203 linked a pull request Jan 29, 2025 that will close this issue
Open
singh1203 added a commit to singh1203/arrow-go that referenced this issue Jan 30, 2025
@singh1203
(GH:apache#58) (ci)(refactor) gofmt check in pre-commit file
63b4432 
Signed-off-by: Saurabh Kumar Singh <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Type: enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

CI: Add gofmt check in lint singh1203/arrow-go
3 participants
@zeroshade @yevgenypats @singh1203