google/gvisor

Use error wrapping in Go 1.13

Open

#2,270 opened on Mar 26, 2020

View on GitHub
 (18 comments) (0 reactions) (0 assignees)Go (1,061 forks)batch import
good first issuepriority: p3status: help wantedtype: cleanup

Repository metrics

Stars
 (12,713 stars)
PR merge metrics
 (Avg merge 4d 22h) (184 merged PRs in 30d)

Description

Make use of the new error wrapping functionality and API to be able to check error types easier.

https://blog.golang.org/go1.13-errors

Contributor guide

Research direction
Identify all places in the codebase where errors are created with fmt.Errorf without %w wrapping, then replace them with wrapped errors using %w and use errors.Is/As for checking.
Tech stack
go
Domain
backendinfrastructure
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
2
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Active
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
Go
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
60