remove duplicate implicit_optional = True line in setup.cfg

[valeriupredoi]

Description

Thanks @zklaus for noticing this 🍺

---

Checklist

It is the responsibility of the author to make sure the pull request is ready to review. The icons indicate whether the item will be subject to the 🛠 Technical or 🧪 Scientific review.

• 🛠 This pull request has a descriptive title and labels
• 🛠 All checks below this pull request were successful

---

To help with the number pull requests:

• 🙏 We kindly ask you to review two other open pull requests in this repository

[codecov]

Codecov Report

Merging #1791 (829358b) into main (4527173) will increase coverage by 0.39%.
The diff coverage is 99.60%.

@@            Coverage Diff             @@
##             main    #1791      +/-   ##
==========================================
+ Coverage   91.11%   91.50%   +0.39%     
==========================================
  Files         203      202       -1     
  Lines       10906    10904       -2     
==========================================
+ Hits         9937     9978      +41     
+ Misses        969      926      -43     

Impacted Files	Coverage Δ
esmvalcore/config/_diagnostics.py	87.83% <ø> (ø)
esmvalcore/config/_logging.py	97.67% <ø> (ø)
esmvalcore/config/_config.py	98.68% <98.68%> (ø)
esmvalcore/_citation.py	80.99% <100.00%> (ø)
esmvalcore/_config/__init__.py	100.00% <100.00%> (ø)
esmvalcore/_data_finder.py	96.69% <100.00%> (ø)
esmvalcore/_main.py	90.66% <100.00%> (+13.31%)	⬆️
esmvalcore/_recipe.py	95.81% <100.00%> (-0.10%)	⬇️
esmvalcore/_task.py	71.55% <100.00%> (-0.68%)	⬇️
esmvalcore/cmor/table.py	94.49% <100.00%> (+0.20%)	⬆️
... and 17 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

[zklaus]

I see we are keeping the reverse alphabetical ordering.

[valeriupredoi]

agh! @bouweandela how do you shut up cov - the line's complaining about is ye olde, and saw that in your PR that just got merged too, but thought it'd keep quiet after merge

[valeriupredoi]

I see we are keeping the reverse alphabetical ordering.

that good or bad? 😁

[bouweandela]

Alphabetical is nicer, but let's get this merged. I have no idea what codecov is thinking here..

[valeriupredoi]

I thought you were the cov whisperer Bouwe 😆

[bouweandela]

Thinking about it: the coverage change from #1769 is probably getting picked up because no coverage report was uploaded when that was merged into main because the tests failed on CircleCI.

[valeriupredoi]

bingo 💡 That's it, man!