modified as per tooling suggestions [changelog skip]

[ignacionr]

Description

This is helping resolve https://github.com/puma/puma/runs/1053004549 that prevent CI from deploying #2314 esp. Rubocop's complaint.

As per the checklist below, this should include a tag to skip CI, but can't do since fixing CI is what it does :)

Your checklist for this pull request

• I have reviewed the guidelines for contributing to this repository.
• I have added an entry to History.md if this PR fixes a bug or adds a feature. If it doesn't need an entry to HISTORY.md, I have added [changelog skip] or [ci skip] to the pull request title.
• I have added appropriate tests if this PR fixes a bug or adds a feature.
• My pull request is 100 lines added/removed or less so that it can be easily reviewed.
• If this PR doesn't need tests (docs change), I added [ci skip] to the title of the PR.
• If this closes any issues, I have added "Closes #issue" to the PR description or my commit messages.
• I have updated the documentation accordingly.
• All new and existing tests passed, including Rubocop.

[nateberkopec]

I don't really want to merge into 4.3.5, it's a dead branch because that release is already out. Is this problem fixed on master/4.3.6?

[ignacionr]

I understand. The company I am working for, is now open to updating their version to 4.3.6, which seems to compile without warnings (warnings were the issue with the recent update of Xcode on the Mac).

Thanks for considering it though.

[ignacionr]

Cancelled request.

[nateberkopec]

Cheers. You made me realize we should be doing 4-3-stable, rails-style branching anyway.