Use an explicit comparison to None for the converter of a field

[filbranden]

Summary

Make attrs work with a converter instance that does not evaluate True. Other code spots already do an explicit comparison to None, so do that here as well. Failing to do so on a valid converter instance that is a valid callable but has a __bool__() that evaluates to False ends up trying to call _fmt_converter_call on that callable rather than on the Converter instance created from it.

Pull Request Check List

• Do not open pull requests from your main branch – use a separate branch!
  • There's a ton of footguns waiting if you don't heed this warning. You can still go back to your project, create a branch from your main branch, push it, and open the pull request from the new branch.
  • This is not a pre-requisite for your pull request to be accepted, but you have been warned.
• Added tests for changed code.
  Our CI fails if coverage is not 100%.
• N/A New features have been added to our Hypothesis testing strategy.
• N/A Changes or additions to public APIs are reflected in our type stubs (files ending in .pyi).
• N/A Updated documentation for changed code.
• N/A Documentation in .rst and .md files is written using semantic newlines.
• Changes (and possible deprecations) have news fragments in changelog.d.
• Consider granting push permissions to the PR branch, so maintainers can fix minor issues themselves without pestering you.

[filbranden]

@hynek Would you kindly take a look whenever possible? Thanks!

[hynek]

Thanks!