Skip to content

Conversation

tobiasraabe
Copy link
Member

Changes

The parametrization was closed and did not allow for customization by plugins. Especially, if we want to convert some argument names in the parametrize decorator to decorators for the generated task functions, we need an entry-point.

The changes are tested against pytask-latex and pytask-r and allow to compile the same document to different formats or execute the same R script with different command line inputs with parametrized tasks.

@tobiasraabe tobiasraabe added this to the v0.0.4 milestone Jul 22, 2020
@codecov
Copy link

codecov bot commented Jul 22, 2020

Codecov Report

Merging #9 into main will decrease coverage by 0.01%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main       #9      +/-   ##
==========================================
- Coverage   94.10%   94.09%   -0.02%     
==========================================
  Files          31       31              
  Lines        1391     1405      +14     
==========================================
+ Hits         1309     1322      +13     
- Misses         82       83       +1     
Flag Coverage Δ
#end_to_end 81.92% <81.57%> (-0.33%) ⬇️
#integration 51.10% <96.05%> (+0.56%) ⬆️
#unit 78.79% <39.47%> (-0.08%) ⬇️

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

Impacted Files Coverage Δ
src/pytask/collect.py 91.59% <ø> (-0.14%) ⬇️
src/pytask/hookspecs.py 100.00% <100.00%> (ø)
src/pytask/parametrize.py 98.61% <100.00%> (-1.39%) ⬇️
tests/test_parametrize.py 95.34% <100.00%> (+0.75%) ⬆️

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 7b124b7...fa45c71. Read the comment docs.

@tobiasraabe tobiasraabe changed the title Refactor parametrization and add hooks. Release 0.0.4 and refactor parametrization and add hooks. Jul 22, 2020
@tobiasraabe tobiasraabe merged commit 82186ef into main Jul 22, 2020
@tobiasraabe tobiasraabe deleted the refactor-parametrization branch July 22, 2020 09:44
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