Skip to content
This repository was archived by the owner on Jan 30, 2025. It is now read-only.

Conversation

inancgumus
Copy link
Collaborator

@inancgumus inancgumus commented Feb 14, 2024

What?

Fixes and bumps the Go versions for the linter and test Github actions.

Why?

Checklist

  • I have performed a self-review of my code
  • I have added tests for my changes
  • I have commented on my code, particularly in hard-to-understand areas

Related PR(s)/Issue(s)

#850, #1214, #1212

@inancgumus inancgumus self-assigned this Feb 14, 2024
@inancgumus inancgumus force-pushed the bump/github-action-go-versions branch from 8d0a559 to 44ad4a0 Compare February 14, 2024 07:13
@inancgumus inancgumus changed the title Bump and fix Github action Go versions Bump linter Github action Go versions Feb 14, 2024
@inancgumus inancgumus force-pushed the bump/github-action-go-versions branch 8 times, most recently from 99b3462 to 2a276f8 Compare February 14, 2024 07:57
@inancgumus inancgumus changed the title Bump linter Github action Go versions Bump linter and test Github action Go versions Feb 14, 2024
@inancgumus inancgumus force-pushed the bump/github-action-go-versions branch from 1849a32 to e65a414 Compare February 14, 2024 08:13
@inancgumus inancgumus requested a review from ankur22 February 14, 2024 08:13
@inancgumus inancgumus marked this pull request as ready for review February 14, 2024 08:13
Copy link
Collaborator

@ankur22 ankur22 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

- Linter wasn't working after #850 change.
- The previous version is no longer Go 1.19.
@inancgumus inancgumus force-pushed the bump/github-action-go-versions branch from e65a414 to b517576 Compare February 14, 2024 10:30
@inancgumus inancgumus merged commit e3fa370 into main Feb 14, 2024
@inancgumus inancgumus deleted the bump/github-action-go-versions branch February 14, 2024 10:39
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants