search

socketry / async-container

Scalable multi-thread multi-process containers for Ruby.
MIT License
81 stars 4 forks source link

Avoid Ruby warning on unused variable #7

Closed olleolleolle closed 4 years ago

olleolleolle commented 4 years ago

This PR avoids a few Ruby warnings which appeared when I ran the test suite.

Most were in the tests, but one was in a lib/ file.

Example from GitHub Actions:

/Users/runner/runners/2.165.2/work/async-container/async-container/spec/async/container/forked_spec.rb:48: warning: assigned but unused variable - child_pid
/Users/runner/runners/2.165.2/work/async-container/async-container/spec/async/container/notify/pipe_spec.rb:41: warning: assigned but unused variable - child
/Users/runner/runners/2.165.2/work/async-container/async-container/lib/async/container/notify/server.rb:101: warning: assigned but unused variable - address
/Users/runner/runners/2.165.2/work/async-container/async-container/lib/async/container/notify/server.rb:101: warning: assigned but unused variable - flags
/Users/runner/runners/2.165.2/work/async-container/async-container/lib/async/container/notify/server.rb:101: warning: assigned but unused variable - controls
coveralls commented 4 years ago

Coverage Status

Coverage remained the same at 79.173% when pulling ca50f8cdd0b9d28fca31a8c6b7e83a431da38668 on olleolleolle:avoid-ruby-warnings-on-unused-var into 45a2f5dab80e21647a14bc79cf1ce869f5f42ebe on socketry:master.