clay / claycli / 39694731768

85%

master: 85%

Pull #275

github

🍕 import: publish after every draft is in, pages last, and exit 1 on errors

`clay import --publish` sent each item's @published PUT right after its own
draft, with `concurrency` items in flight. `clay export` emits a page's
components and layout before the page, so a page publish could run while
its components were still being written. Amphora's page publish reads every
draft the page composes, so it answered "Not Found" or "Bad Request" and the
page stayed unpublished, while the CLI still exited 0.

Now every draft is written first, and the queued publishes run once the
drafts have drained: components and layouts first, then pages, so a page
publish is also the last write to its components' @published data. An item
whose own draft failed is not published. The CLI sets exit code 1 when any
item reports an error.

939 of 1204 branches covered (77.99%)

Branch coverage included in aggregate %.

21 of 21 new or added lines in 1 file covered. (100.0%)

1902 of 2148 relevant lines covered (88.55%)

27.08 hits per line