fix: fix old app version deploy wrt. compose files #411

Merged
decentral1se merged 1 commits from versions-checkout-fix into main 2024-06-21 14:26:56 +00:00
Owner

See coop-cloud/organising#617

Manually ran the test suite on app_deploy.bats and things seem to be in order!

See https://git.coopcloud.tech/coop-cloud/organising/issues/617 Manually ran the test suite on `app_deploy.bats` and things seem to be in order!
decentral1se requested review from simon 2024-06-19 18:33:05 +00:00
p4u1 reviewed 2024-06-20 21:22:44 +00:00
@ -373,0 +397,4 @@
assert_success
refute_output --partial 'no such file or directory'
_undeploy_app
Member

Would be nice to get the cleanup simplified in the future....
Something like _reset_all

Would be nice to get the cleanup simplified in the future.... Something like `_reset_all`
Author
Owner

Yeh this remains quite an elusive thing... a cleaning up of order of execution, cleanup, etc.

I will make a ticket 🙃

Yeh this remains quite an elusive thing... a cleaning up of order of execution, cleanup, etc. I will make a ticket 🙃
decentral1se marked this conversation as resolved
Member

Can we now remove the same block starting on line 131?

Can we now remove the same block starting on line 131?
Author
Owner

Can we now remove the same block starting on line 131?

Good shout, I can just move that block up instead of duplicating it 😌 Thanks!

> Can we now remove the same block starting on line 131? Good shout, I can just move that block up instead of duplicating it 😌 Thanks!
decentral1se force-pushed versions-checkout-fix from cfbafd2613 to 91ea2c01a5 2024-06-21 14:26:27 +00:00 Compare
decentral1se merged commit 91ea2c01a5 into main 2024-06-21 14:26:56 +00:00
decentral1se deleted branch versions-checkout-fix 2024-06-21 14:26:56 +00:00
Sign in to join this conversation.
No description provided.