Skip to content

chore: pin mdref to v0.5.1#1430

Merged
hilmarf merged 1 commit into
mainfrom
pin/mdref
Apr 29, 2025
Merged

chore: pin mdref to v0.5.1#1430
hilmarf merged 1 commit into
mainfrom
pin/mdref

Conversation

@hilmarf
Copy link
Copy Markdown
Member

@hilmarf hilmarf commented Apr 29, 2025

@mandelsoft created a new tag: https://github.com/mandelsoft/mdref/releases/tag/v0.5.1 which doesn't result in sha-sum mismatches.

What this PR does / why we need it

We should not use the ugly workaround with the GONOSUMDB setting.

Which issue(s) this PR fixes

fixes: mandelsoft/mdref#1

@hilmarf hilmarf requested a review from a team as a code owner April 29, 2025 07:37
@hilmarf hilmarf enabled auto-merge (squash) April 29, 2025 07:37
@github-actions github-actions Bot added kind/chore chore, maintenance, etc. size/xs Extra small labels Apr 29, 2025
Copy link
Copy Markdown
Contributor

@frewilhelm frewilhelm left a comment

Choose a reason for hiding this comment

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

This pin will be recognised by dependabot, right?

@hilmarf
Copy link
Copy Markdown
Member Author

hilmarf commented Apr 29, 2025

This pin will be recognised by dependabot, right?

nope, I don't think so. I haven't seen any samples where Dependabot automatically checked Makefiles... That would require some custom dependabot config.

@hilmarf hilmarf merged commit eb496f0 into main Apr 29, 2025
26 checks passed
@hilmarf hilmarf deleted the pin/mdref branch April 29, 2025 07:57
@ocmbot ocmbot Bot added this to the 2025-Q2 milestone Apr 29, 2025
Comment thread hack/Makefile
mdref:
# https://github.com/mandelsoft/mdref/issues/1 as workaround set GONOSUMDB
GONOSUMDB='github.com/mandelsoft/mdref' go install -v github.com/mandelsoft/mdref@v0.5.0
go install -v github.com/mandelsoft/mdref@v0.5.1
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It used to pin @master before failure. Do we want to stick to a specific version now?

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

Labels

kind/chore chore, maintenance, etc. size/xs Extra small

Projects

None yet

Development

Successfully merging this pull request may close these issues.

go SECURITY ERROR - checksum mismatch

4 participants