Skip to content

Fix for go-1.18, Errorf doesn't accept verb %w#58

Open
mikelolasagasti wants to merge 1 commit intomwitkow:masterfrom
mikelolasagasti:fix-go118
Open

Fix for go-1.18, Errorf doesn't accept verb %w#58
mikelolasagasti wants to merge 1 commit intomwitkow:masterfrom
mikelolasagasti:fix-go118

Conversation

@mikelolasagasti
Copy link

No description provided.

@dbudworth
Copy link

https://pkg.go.dev/fmt#Errorf

%w not accepted in 1.18?
https://go.dev/play/p/KztaAZ-xjmT
indicates it does, what was the problem you ran in to?

@mikelolasagasti
Copy link
Author

The error I get is:

./testping.go:102:5: (*testing.common).Errorf does not support error-wrapping directive %w

@dbudworth
Copy link

oops, my mistake. For some reason I read this as fmt.Errorf, not t.Errorf

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.

2 participants