search

MolecularAI / aizynthfinder

A tool for retrosynthetic planning
https://molecularai.github.io/aizynthfinder/
MIT License
562 stars 128 forks source link

Use default_factory for inner data class #116

Closed chajath closed 7 months ago

chajath commented 1 year ago

In python 3.11, it will fail otherwise

SGenheden commented 1 year ago

Thanks for this. We became aware of this bug a few days ago.

codecov-commenter commented 1 year ago

Codecov Report

Merging #116 (e3a4e05) into master (ead1aa2) will not change coverage. The diff coverage is 100.00%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##           master     #116   +/-   ##
=======================================
  Coverage   83.49%   83.49%           
=======================================
  Files          65       65           
  Lines        4927     4927           
  Branches      810      810           
=======================================
  Hits         4114     4114           
  Misses        620      620           
  Partials      193      193
Impacted Files Coverage Δ
aizynthfinder/context/config.py 95.91% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

SGenheden commented 7 months ago

Fixed in #141