fix: unstaged changes handling #460

Merged
decentral1se merged 1 commits from unstaged-refactor into main 2024-12-31 15:38:56 +00:00
Owner

toolshed/organising#651

Motivated by brainmelt of coop-cloud/abra#458 😂

In the spirit of: "show all information for people to make a decision" vs. "try to figure out what they are doing" - it is less complex in the code and supports more workflows!

So far, we have:

  • write tests
  • store +U in app.Recipe.Dirty
  • query recipe dirty status with Recipe.IsDirty()
  • write +U to .env (already implemented)
  • parse +U into Recipe.Dirty on read
  • ensure +U carried into deployment labels
  • +U shows in bold on all overviews
  • reworked all overviews for moar overview
  • String() for core types which improves log output
  • remove redundant logging
  • abra app labels to query the labels (local/remote)!
image

On the way, I fixed 2 bonus bugs:

  • abra app ps reads chaos-version label for chaos version
  • release notes are shown in correct descending order

CAN'T RESIST

image

(I was tempted by app.Recipe.ChristinaAguilera but alas...)

https://git.coopcloud.tech/toolshed/organising/issues/651 Motivated by brainmelt of https://git.coopcloud.tech/coop-cloud/abra/pulls/458 😂 In the spirit of: "show all information for people to make a decision" vs. "try to figure out what they are doing" - it is less complex in the code and supports more workflows! So far, we have: - [x] write tests - [x] store `+U` in `app.Recipe.Dirty` - [x] query recipe dirty status with `Recipe.IsDirty()` - [x] write `+U` to `.env` (already implemented) - [x] parse `+U` into `Recipe.Dirty` on read - [x] ensure `+U` carried into deployment labels - [x] `+U` shows **in bold** on all overviews - [x] reworked all overviews for *moar overview* - [x] `String()` for core types which improves log output - [x] remove redundant logging - [x] `abra app labels` to query the labels (local/remote)! <img width="292" alt="image" src="attachments/3b3731d6-f76c-4b5d-a0a8-01523ff22ab2"> On the way, I fixed 2 bonus bugs: - [x] `abra app ps` reads `chaos-version` label for chaos version - [x] release notes are shown in correct descending order **CAN'T RESIST** ![image](/attachments/18835dce-a268-4d62-8e76-f31efb3cc553) (I was tempted by `app.Recipe.ChristinaAguilera` but alas...)
decentral1se force-pushed unstaged-refactor from 40a17f1175 to 55d69fa0a5 2024-12-31 10:52:48 +00:00 Compare
decentral1se force-pushed unstaged-refactor from 55d69fa0a5 to 243430fdce 2024-12-31 11:50:11 +00:00 Compare
decentral1se force-pushed unstaged-refactor from 243430fdce to 5975be6870 2024-12-31 15:37:12 +00:00 Compare
decentral1se changed title from WIP: fix: unstaged changes handling to fix: unstaged changes handling 2024-12-31 15:38:15 +00:00
decentral1se merged commit 5975be6870 into main 2024-12-31 15:38:56 +00:00
decentral1se deleted branch unstaged-refactor 2024-12-31 15:38:56 +00:00
Sign in to join this conversation.
No description provided.