Mateusz Hawrus

Results 45 comments of Mateusz Hawrus

hey @kenfinnigan, I'll close this one but I will use some parts of it to create a PR covering the same ground, thanks for the effort here!

This looks outdated given the current spec, feel free to reopen If you feel differently.

I agree, there are similar issues already: https://github.com/gruntwork-io/terratest/issues/1193#issuecomment-1434782474

I was also totally confused by that, but, alas, there's a better way :) Do this instead: ```json { "gopls": { "ui.diagnostic.analyses": { "fieldalignment": true, "nilness": true, "unusedparams": true, "unusedwrite":...

@denis256 how is this fixing the issue? You will still run `helm dependency build` each and every time, you shouldn't be making that decision for your users in first place,...

Furthermore I noticed it also removes dependencies, which is why I just copy the scaffold and use `sed`. At the current state it doesn't seem to be very helpful tbh....

bumping this up, any news on the possible adoption?

it also contradicts the `encoding/json` behaviour, one workaround for that I found is to use `yaml.UseJSONUnmarshaler()`, but this is not ideal obviously :/ would you consider a contribution to change...

maybe it would be enough to simply export `errors.syntaxError`? This way we can use `errors.As` to get to the root syntax error.

Coming to think about, `TypeError` is equally useful as `SyntaxError`. Maybe we could simply export the whole package? Would you consider that @goccy? Then we wouldn't need to implement any...