Skip to content

Stop using github.com/pkg/errors#555

Merged
openshift-merge-bot[bot] merged 5 commits intocontainers:mainfrom
cfergeau:no-wrap
Sep 24, 2025
Merged

Stop using github.com/pkg/errors#555
openshift-merge-bot[bot] merged 5 commits intocontainers:mainfrom
cfergeau:no-wrap

Conversation

@cfergeau
Copy link
Copy Markdown
Collaborator

This has been deprecated since go 1.13 introduced %w and
errors.Is/errors.As

Copy link
Copy Markdown
Collaborator

@lstocchi lstocchi left a comment

Choose a reason for hiding this comment

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

LGTM, just a couple of suggestions

This is deprecated in favour of `%w` use.

Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
@lstocchi
Copy link
Copy Markdown
Collaborator

/lgtm
/approve

@openshift-ci
Copy link
Copy Markdown
Contributor

openshift-ci bot commented Sep 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: cfergeau, lstocchi

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-merge-bot openshift-merge-bot bot merged commit 9e98ea7 into containers:main Sep 24, 2025
21 of 22 checks passed
@cfergeau cfergeau deleted the no-wrap branch September 25, 2025 07:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants