Riccardo Magliocchetti

Results 120 comments of Riccardo Magliocchetti

@naga2222 Thanks for the PR, could you please elaborate a bit on why the change is needed?

> Do we really need this? 🤔 We already have linting for every package, is there a place where we are not providing linting right now? The only reason for...

Why is that a problem? Making it explicit is only adding more churn when updating test requirements in my opinion.

@pragathi0107 jinja dependency is already at latest version, I don't think there's anything else to change regarding it.

@samuelcolvin any chance you could take a look at this?

@Kludex it would be great if you can take a look at this, thanks

@VamsiKrishna1211 care to add a test with updated test requirements so we can actually test it in CI?

> @xrmx Yes sure. This is my first contribution to otel-contrib repository. So please let me know if I have to make any other changes in the PR. :) Cool!...

@VamsiKrishna1211 you can take a look at `opentelemetry-instrumentation-httpx` both in tox.ini (grep for httpx) and in the instrumentation package dir

@VamsiKrishna1211 could you please add a CHANGELOG entry? And fix lint and CLA please