Skip to content

Remove leftover fmt.Errorf() after merging shift width check - #53

Merged
arthurfabre merged 1 commit into
masterfrom
afabre/merge
Aug 28, 2026
Merged

Remove leftover fmt.Errorf() after merging shift width check#53
arthurfabre merged 1 commit into
masterfrom
afabre/merge

Conversation

@arthurfabre

@arthurfabre arthurfabre commented Aug 28, 2026

Copy link
Copy Markdown
Collaborator

I merged two PRs successively, and broke builds:

  • eaafb3a "Reject cBPF shifts > 31" added a new errors.Errorf() call.
  • 71323da "Replace github.com/pkg/errors with stdlib" dropped the errors package.

Change the added errors.Errorf() call over to fmt.Errorf() too.

@arthurfabre
Remove leftover fmt.Errorf() after merging shift width check
3d7a663 
I merged two PRs successively, and broke builds:

- eaafb3a "Reject cBPF shifts > 31" added a new errors.Errorf() call.
- 71323da "Replace github.com/pkg/errors with stdlib" dropped the
  errors package.

Change the added errors.Errorf() call over to fmt.Errorf() too.
@arthurfabre
arthurfabre requested a review from Dhiver August 28, 2026 08:37
@arthurfabre arthurfabre self-assigned this Aug 28, 2026
@arthurfabre
arthurfabre merged commit 3992b6e into master Aug 28, 2026
10 checks passed
@arthurfabre
arthurfabre deleted the afabre/merge branch August 28, 2026 09:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@Dhiver Dhiver Dhiver approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@arthurfabre arthurfabre

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@arthurfabre @Dhiver