Pablo Baeyens

Results 291 comments of Pablo Baeyens

There are only three components left. In particular: - ~~A PR seems to have accidentally reverted the Jaeger receiver errcheck enabling (see https://github.com/open-telemetry/opentelemetry-collector-contrib/pull/14561#discussion_r994550546 )~~ FIXED. - ~~Recent PRs to the...

Indeed :tada: :rocket: Closing...

``` ❯ rg -tgo gocritic -l | cut -d '/' -f 1-2 | sort | uniq pkg/ottl receiver/awscontainerinsightreceive ``` These are the two components missing: - The first one has...

Indeed :tada: :tada: Thanks for your hard work on this @fatsheep9146 !

I also think being consistent is important. I don't feel like this complicates much the process of adding a changelog entry, but maybe there is some issue I am not...

It can also help us avoid easy mistakes like adding the changelog entry to the wrong part of the changelog document. For example #6052 change is added to 0.59.0. Currently...

In case it's useful, it looks like exa doesn't handle `TZ` correctly on Fedora 35. It falls under the case *"When TZ is an absolute path (starts with /)"* +...

Any chance this will be reopened? As mentioned above, there is widespread support for hyperlinks on modern terminal emulators and tools like `ls` have built-in support for this.

`gopls` also relies on the Go parser, they had a similar-ish issue (golang/go#41057) and fixed it (golang/tools@cf7880770c1e80a90b5973b87c1583154b05af5f) while still relying on the Go parser. I was hoping a similar thing...

This can still be reproduced with the latest version (`main.go` being the file on my original comment): ``` ❯ go install github.com/mgechev/revive@latest go: downloading github.com/mgechev/revive v1.2.3 go: downloading github.com/chavacava/garif v0.0.0-20220630083739-93517212f375...