Skip to content

annotate with Sequence rather than list & avoid mutable defaults#43

Merged
MLopez-Ibanez merged 1 commit intomulti-objective:mainfrom
tpgillam:tg/sequence
Nov 29, 2025
Merged

annotate with Sequence rather than list & avoid mutable defaults#43
MLopez-Ibanez merged 1 commit intomulti-objective:mainfrom
tpgillam:tg/sequence

Conversation

@tpgillam
Copy link
Copy Markdown
Contributor

Allow users to specify e.g. tuples of values rather than lists (this previously works at runtime, but statically the types didn't allow it).

Also remove a couple of cases of default arguments being mutable.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 29, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.45%. Comparing base (2c66c8a) to head (84ac8de).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
+ Coverage   73.39%   73.45%   +0.05%     
==========================================
  Files          58       58              
  Lines        5041     5041              
  Branches      964      964              
==========================================
+ Hits         3700     3703       +3     
+ Misses        964      962       -2     
+ Partials      377      376       -1     
Flag Coverage Δ
python 72.54% <100.00%> (+0.06%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Components Coverage Δ
tests 100.00% <ø> (ø)
src 86.99% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@MLopez-Ibanez MLopez-Ibanez merged commit 6330891 into multi-objective:main Nov 29, 2025
23 checks passed
@MLopez-Ibanez
Copy link
Copy Markdown
Contributor

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants