Repository navigation

Remove naked return values from `ReadBranchConfig` and `prSelectorForCurrentBranch` - #10197

Merged

jtmcg merged 19 commits into

trunkfrom

jtmcg/remove-named-returns

Jan 13, 2025

Merged

Remove naked return values from `ReadBranchConfig` and `prSelectorForCurrentBranch`#10197
jtmcg merged 19 commits into
trunkfrom
jtmcg/remove-named-returns

Commits on Jan 7, 2025
- Remove named returns from ReadBranchConfig and surface errors
  jtmcg
  committed
  bf6fdbd
- Refactor ReadBranchConfig for test coverage of newly returned erros
  jtmcg
  committed
  f43da8e
- Remove named return values from prSelectorForCurrentBranch
  jtmcg
  committed
  c83cf32
- Refactor Test_readGitBranchConfig for easier parsing
  jtmcg
  committed
  0137150
Commits on Jan 8, 2025
Commits on Jan 9, 2025
- Add test for empty BranchConfig in prSelectorForCurrentBranch
  jtmcg
  committed
  d4f7576
- Refine error handling of ReadBranchConfig
  jtmcg
  committed
  e1423cd
- Refactor error handling in ReadBranchConfig to avoid panic
  jtmcg
  committed
  48d45d0
- Directly stub headBranchConfig in Test_tryDetermineTrackingRef
  jtmcg
  committed
  e4320cc
- Surface and handle error from ReadBranchConfig in parseCurrentBranch
  jtmcg
  committed
  e4f0b79
- Change pr number in test
  jtmcg
  committed
  100f9b4
- Rename test user in tests
  jtmcg
  committed
  3d27487
Commits on Jan 10, 2025
- Change error handling on ReadBranchConfig to respect git Exit Codes
  jtmcg
  committed
  322a43f
- Restore old error functionality of prSelectorForCurrentBranch
  jtmcg
  committed
  74cbd50