chore: 🧑💻 skip raw tasks if API key is empty - #107
lwjohnst86 wants to merge 10 commits into
Conversation
1755c32 to
0a16217
Compare
|
|
||
|
|
||
| @mark.metadata | ||
| @mark.skipif( |
There was a problem hiding this comment.
Was it the intention to make the metadata tasks run without the download step? If I run uv run pytask build -m metadata it will skip all tasks now because -- I think -- it still can't see that this first dependency is satisfied (rather, the file's existence doesn't mean that the dependency is satisfied). Or is it working for you?
There was a problem hiding this comment.
Hmmm, I hadn't noticed that it wasn't working for me, but I wasn't specifically checking how you describe it. It might need a @mark.persist tag...?
There was a problem hiding this comment.
Yeah, I played around with it a bit and I think @mark.persist might actually be better than skip here. At least, with just @mark.persist it skips the download task but runs the processing tasks.
Then there is also the older idea of having the download task in a separate pipeline with no pytask dependency between the download step and the other steps.
Co-authored-by: martonvago <57952344+martonvago@users.noreply.github.com>
This branch has not been deployed
2 participants
Description
To make local development easier.
Needs a quick review.
Checklist
just run-alljust build-package