Jack Lindamood

Above looks right. Bullet (3) is probably an enhancement off the core request: bullet (1). Also the difference between `last read` and `last modified` is pretty huge in both use...

Why can't the error type have a `Stack() []uintptr` function, error type have a Formatter method, and there be helpers to format `[]uintptr` stacks that error uses in its Formatter...

Can't you make `fmt.Printf("%+v", err)` work like it does today by having `func (e _error) Format` format the []uintptr returned by e.Stack() ? Maybe i'm missing something: ``` func (e...

I think I see what you mean now. I don't think it's too much to, if someone wants to format the stack only, to get it out as an []uintptr...

The way the code is now, all the stack traces of wraps get put into the output. Isn't that excessive? > If you interact with a package from another repository,...

> I can equally see a need too add a strack trace for each wrap Don't you agree that's strongly overstated? Yes, the two situations exist, but to claim they...

It may be worth reading https://github.com/pkg/errors/issues/34

An explicit `null` is very different than not being present. For example, with helm charts you can explicitly set values to `null` that are part of the default `values.yaml`. It's...

Here is the expected output: ``` apiVersion: helm.toolkit.fluxcd.io/v2beta1 kind: HelmRelease metadata: name: chart spec: releaseName: helm-chart timeout: 15m values: chart: autoscaling: true replicaCount: null deepgram-api: some: value ```

> What is the type of values.chart.replicaCount in your CR? It's a somewhat common pattern in helm charts. There's a `replicaCount` that has some default value, like 1. And if...