Deprecate bare connector close

[asvetlov]

Use async with connector: and await connector.close() instead

[codecov-io]

Codecov Report

Merging #3417 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master    #3417      +/-   ##
==========================================
+ Coverage   97.94%   97.95%   +<.01%     
==========================================
  Files          44       44              
  Lines        8522     8543      +21     
  Branches     1383     1383              
==========================================
+ Hits         8347     8368      +21     
  Misses         74       74              
  Partials      101      101

Impacted Files	Coverage Δ
aiohttp/helpers.py	97.2% <100%> (+0.01%)	⬆️
aiohttp/connector.py	97.67% <100%> (+0.07%)	⬆️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a0764c5...a2ea6ef. Read the comment docs.

[webknjaz]

Let's rename it to represent the intent:

Suggested change

	async def noop2(*args: Any, **kwargs: Any) -> None:
	async def async_noop(*args: Any, **kwargs: Any) -> None:

[asvetlov]

1. Both noops are async.
2. I don't care too much, the function is internal and will be removed in aiohttp 4.

[lock]

This thread has been automatically locked since there has not been
any recent activity after it was closed. Please open a new issue for
related bugs.

If you feel like there's important points made in this discussion,
please include those exceprts into that new issue.