search

dnephin / dobi

A build automation tool for Docker applications
https://dnephin.github.io/dobi/
Apache License 2.0
309 stars 36 forks source link

Cleanup/refactor actions #228

Open aidan-mundy opened 2 years ago

aidan-mundy commented 2 years ago

Depends on #223 (includes commits from that PR) Depends on #226 (includes commits from that PR) Relates to #227

This is a large refactor of how actions/dependencies are handled. It resolves item 2 in #227, and as far as I can tell it also resolves item 1 in #227 (prior to these updates, dependencies would sometimes be built before deleting the "depended")

NOTE: Marked as draft for now because resolving items 1&2 causes item 3 to be much more noticeable.

aidan-mundy commented 2 years ago

A future optimization to these changes would be to frontload action parsing during the config reading process and store Name items instead of raw strings, instead of passing errors all over the place if dependency parsing fails later.