Skip to content

chore: replace github.com/pkg/errors with native error package in the acl, algo, audit, backup, check_upgrade and chunker modules #9372

Merged
mangalaman93 merged 1 commit intodgraph-io:mainfrom
xqqp:remove_pkg_errors
Apr 16, 2025
Merged

Conversation

@xqqp
Copy link
Copy Markdown
Contributor

@xqqp xqqp commented Apr 8, 2025 •
edited
Loading

Description

Replaced the github.com/pkg/errors package with the native errors package in the acl, algo, audit, backup, check_upgrade and chunker modules. Also fixed some error comparisons and error texts.

Checklist

  • Code compiles correctly and linting passes locally
  • For all code changes, an entry added to the CHANGELOG.md file describing and linking to
    this PR
  • Tests added for new functionality, or regression tests for bug fixes added as applicable
  • For public APIs, new features, etc., PR on
    docs repo staged and linked here

@xqqp xqqp changed the title chore: replace github.com/pkg/errors with native error package chore: replace github.com/pkg/errors with native error package for for acl, algo, audit, backup, check_upgrade and chunker modules Apr 8, 2025
@xqqp xqqp changed the title chore: replace github.com/pkg/errors with native error package for for acl, algo, audit, backup, check_upgrade and chunker modules chore: replace github.com/pkg/errors with native error package for acl, algo, audit, backup, check_upgrade and chunker modules Apr 8, 2025
@xqqp xqqp marked this pull request as ready for review April 8, 2025 17:57
@xqqp xqqp requested a review from a team April 8, 2025 17:57
@xqqp xqqp changed the title chore: replace github.com/pkg/errors with native error package for acl, algo, audit, backup, check_upgrade and chunker modules chore: replace github.com/pkg/errors with native error package in the acl, algo, audit, backup, check_upgrade and chunker modules Apr 8, 2025
@mangalaman93 mangalaman93 merged commit 41c9811 into dgraph-io:main Apr 16, 2025
21 of 23 checks passed
@xqqp xqqp deleted the remove_pkg_errors branch April 16, 2025 13:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@mangalaman93 mangalaman93 mangalaman93 approved these changes

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@xqqp @mangalaman93