Skip to content

Conversation

tobiasraabe
Copy link
Member

@tobiasraabe tobiasraabe commented Feb 25, 2022

Changes

  • Add _pytask.models.CollectionMetadata to carry over information about functions from parametrizations to the collection.
  • Add a new attribute to the task class named kwargs which contains information which need to be passed to the task.

@tobiasraabe tobiasraabe added this to the v0.2.0 milestone Feb 25, 2022
@codecov
Copy link

codecov bot commented Feb 25, 2022

Codecov Report

Merging #227 (f854d7c) into main (743ebd1) will increase coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #227      +/-   ##
==========================================
+ Coverage   94.21%   94.23%   +0.01%     
==========================================
  Files          75       76       +1     
  Lines        6761     6764       +3     
==========================================
+ Hits         6370     6374       +4     
+ Misses        391      390       -1     
Flag Coverage Δ
end_to_end 79.36% <93.10%> (+0.11%) ⬆️
integration 41.60% <62.06%> (+0.01%) ⬆️
unit 70.72% <93.10%> (+0.11%) ⬆️

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

Impacted Files Coverage Δ
src/_pytask/execute.py 96.38% <100.00%> (+0.18%) ⬆️
src/_pytask/hookspecs.py 100.00% <100.00%> (ø)
src/_pytask/models.py 100.00% <100.00%> (ø)
src/_pytask/nodes.py 99.29% <100.00%> (-0.02%) ⬇️
src/_pytask/parametrize.py 100.00% <100.00%> (ø)
tests/test_parametrize.py 98.29% <100.00%> (+0.45%) ⬆️

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 743ebd1...f854d7c. Read the comment docs.

@tobiasraabe tobiasraabe merged commit d93d21a into main Feb 25, 2022
@tobiasraabe tobiasraabe deleted the func-kwargs-to-task-kwargs branch February 25, 2022 23:54
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.

1 participant