Skip to content

Conversation

@ArionMiles
Copy link

No description provided.

Copy link
Owner

@nikhita nikhita left a comment

Choose a reason for hiding this comment

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

@ArionMiles sorry for the delaying in responding to this PR, and thanks for the improvements, really appreciate it! :)

@@ -0,0 +1,12 @@
module github.com/nikhita/github-contrib

go 1.11
Copy link
Owner

Choose a reason for hiding this comment

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

Can you move this directly to 1.13 since that's the latest version?

Copy link
Author

Choose a reason for hiding this comment

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

Isn't the go directive here used to specify the minimum go version?

I know you mention minimum go version as >=1.7 but for those using go<1.11, they'll not have any use for go.mod since it's not a feature in those versions.

So I thought adding go1.11 would be a good idea since people with those versions can still use the project.

@ArionMiles
Copy link
Author

I've removed go 1.11 specific instructions but I'm unsure whether updating go.mod to include 1.13 is a good idea, do let me know what you think!

@demoManito
Copy link

I've removed go 1.11 specific instructions but I'm unsure whether updating go.mod to include 1.13 is a good idea, do let me know what you think!

I think it can be updated to 1.19

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants