Skip to content

Add go.mod#36

Merged
thaJeztah merged 1 commit intodocker:mainfrom
thaJeztah:gomod
Apr 10, 2026
Merged

Add go.mod#36
thaJeztah merged 1 commit intodocker:mainfrom
thaJeztah:gomod

Conversation

@thaJeztah
Copy link
Copy Markdown
Member

@thaJeztah thaJeztah commented Aug 4, 2020

Keeping the go version to 1.20 for now, as this module doesn't require "latest and greatest"

Copy link
Copy Markdown
Contributor

@kolyshkin kolyshkin left a comment

Choose a reason for hiding this comment

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

LGTM

@kolyshkin kolyshkin mentioned this pull request Jun 6, 2022
Comment thread go.mod Outdated
@kolyshkin
Copy link
Copy Markdown
Contributor

kolyshkin commented Sep 19, 2024

@thaJeztah Can we revive this, and use (at least) go 1.20 so we can use strings.Cut?

Keeping the go version to 1.20 for now, as this module
doesn't require "latest and greatest"

Signed-off-by: Sebastiaan van Stijn <github@gone.nl>
@thaJeztah
Copy link
Copy Markdown
Member Author

Ah! Thx Akihiro; I'll work on adding some basic actions on this repo as well

@thaJeztah thaJeztah merged commit c614bfb into docker:main Apr 10, 2026
@thaJeztah thaJeztah deleted the gomod branch April 10, 2026 18:15
@kolyshkin
Copy link
Copy Markdown
Contributor

I think we never merged this one because we want to move this to under moby.

Otherwise, I've proposed adding go.mod in PR #40 but had to remove it; see discussion here:

and this issue:

@thaJeztah
Copy link
Copy Markdown
Member Author

Oh! Yeah, you're right!

I'd still like to move it; that said, from some other cases, I think ultimately it doesn't make a big different to have the go.mod; go modules will be confused in either case; try to redirect then either finds a go.mod with a different name, or doesn't, so perhaps it's fine to already have one, then tag -> rename -> tag with new name

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