refactor!: rework recipe release flow #764

Merged
iexos merged 4 commits from iexos/abra:push-xttokwwwvllz into main 2026-02-16 09:18:42 +00:00
Member

closes #682

  • Tests have been adapted/added and are passing
  • It is missing a production test, will do that before merging
  • Docs are in progress
closes #682 * Tests have been adapted/added and are passing * It is missing a production test, will do that before merging * Docs are in progress
iexos added 4 commits 2026-02-13 11:17:28 +00:00
decentral1se approved these changes 2026-02-13 11:57:42 +00:00
decentral1se left a comment
Owner

Looking great, tysm @iexos! I took a quick pass... as discussed in the abra hackers chat, we need some docs and probably an RC to get people testing this. Thanks for writing the tests also 👏

Looking great, tysm @iexos! I took a quick pass... as discussed in the abra hackers chat, we need some docs and probably an RC to get people testing this. Thanks for writing the tests also 👏
decentral1se changed title from Rework recipe release flow to refactor: rework recipe release flow 2026-02-13 11:57:58 +00:00
decentral1se changed title from refactor: rework recipe release flow to refactor!: rework recipe release flow 2026-02-13 11:58:04 +00:00
p4u1 reviewed 2026-02-13 12:04:41 +00:00
@ -93,21 +97,209 @@ your private key and enter your passphrase beforehand.
log.Fatal(i18n.G("main app service version for %s is empty?", recipe.Name))
}
tags, err := recipe.Tags()
Owner

This could be moved below the isClean check

This could be moved below the isClean check
Author
Member

👍 done

👍 done
iexos marked this conversation as resolved
iexos force-pushed push-xttokwwwvllz from 1239780da7 to 4d075a6bb4 2026-02-13 14:16:19 +00:00 Compare
Owner

@iexos please merge this when you're happy with it 👏

@iexos please merge this when you're happy with it 👏
iexos merged commit 521f5c1647 into main 2026-02-16 09:18:42 +00:00
iexos deleted branch push-xttokwwwvllz 2026-02-16 09:18:43 +00:00
Sign in to join this conversation.
No description provided.