the errors should have a better spelling
Open
kind/enhancement
- Dominant language
- Swift
- Stars
- 503
- Forks
- 103
- Avg merge
- 18h 8m
- Merged PRs (30d)
- 4
Description
Currently, the error look like this:
NIOHTTP2Errors.NoSuchStream(...)
but I think it should be either
NIOHTTP2Error.noSuchStream(...) // imitating an open enum on top of a struct
or
NIOHTTP2Errors.NoSuchStreamError(...)
IMHO, preferably the former
Contributor guide
Research direction
Start by locating the NIOHTTP2Errors.NoSuchStream declaration and all of its uses. Check how the existing error cases are exposed and tested, then apply the preferred NIOHTTP2Error.noSuchStream spelling consistently; done means the public error API and its references use the agreed naming.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- swift
- Domain
- networking
- Issue type
- Refactor
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100