Joseph Lombrozo

This way we can do things like: - require the check before merging - show "in progress" while we're building - show commit success/failure

Generate hosted html summaries and details

3

When adding or modifying large applications, the PR/MR comment often overflows and you lose data. If we were to generate and store html summaries, and link to them via the...

add support for github app auth

2

Closes https://github.com/zapier/kubechecks/issues/247. Builds on top of https://github.com/zapier/kubechecks/pull/249. Also gracefully handles missing comment permissions Adds the following env vars: |Env Var|Description|Default Value| |-----------|-------------|------| |`KUBECHECKS_GITHUB_APP_ID`|Github App ID.|`0`| |`KUBECHECKS_GITHUB_INSTALLATION_ID`|Github Installation ID.|`0`| |`KUBECHECKS_GITHUB_PRIVATE_KEY`|Github App...

Document the different ways of avoiding TLS requirements

2

Scenarios that I can think of: - ArgoCD doesn't have a valid certificate - ArgoCD is listening via plaintext It seems that recent changes have necessitate setting an additional environment...

It provides no value and increases the diff size by a ton.

Turn on automatic retries for sandbox requests

1

> [!NOTE] > Adds up to 3 automatic retries when sending sandbox PTY input, retrying on common transient/network errors. > > - **CLI – `packages/cli/src/terminal.ts`** > - **Retry logic**: Wraps...

This helps us break apart the main function into components, which also helps us close cleanly. --- > [!NOTE] > Refactors the orchestrator to an Uber FX-based modular app with...

Add the 'early-return' linter, pass more ctx

1

Also tweak some integration tests to try and make them less flaky. Success? --- > [!NOTE] > Enable the early-return linter and refactor code and tests to use early returns...