envoyproxy / envoyproxy/nighthawk
Make the SignalHandler class implementation easier to follow
Open
enhancement
good first issue
- Dominant language
- C++
- Stars
- 414
- Forks
- 95
- Avg merge
- 2d 9h
- Merged PRs (30d)
- 11
Description
The SignalHandler class could use a little love (code level docs, member renaming for clarity), it requires too much brainpower to follow as it stands. @eric846's [review comment](https://github.com/envoyproxy/nighthawk/pull/602/files#r554281486) looks like a great starting point for a class level description.
Contributor guide
Assessment
This issue has not been assessed yet.