Darko Janjic

Fixes #13465 ### Motivation Reduce unneeded nesting by changing `if assert.*` to `require.*`. ### Modifications Updated tests where `if assert.*` is used to `require.*` ### Verification Tests

### Pre-requisites - [x] I have double-checked my configuration - [x] I have tested with the `:latest` image tag (i.e. `quay.io/argoproj/workflow-controller:latest`) and can confirm the issue still exists on `:latest`....

`make devcontainer-up` fails if `devcontainer` is already installed using `homebrew`

2

### Pre-requisites - [x] I have double-checked my configuration - [x] I have tested with the `:latest` image tag (i.e. `quay.io/argoproj/workflow-controller:latest`) and can confirm the issue still exists on `:latest`....

# Summary `kmsId` and `nodeId` fields are excluded from linting (ST1003) since this change affects generated files (like `proto` and `swagger`) and is a breaking change. Also, changing these files...