Pranav Marla

Results 24 comments of Pranav Marla

@srikanthccv I attempted to fix all the failing workflow checks that I could, by updating the changelog and linting the code. (The `docker-tests` check seems to have failed due to...

@srikanthccv Done! **Update**: For clarity, added to summary of changes in original post

Update: - Linted example - Ran new spellcheck check to be safe and updated PR description with results

Unfortunately, pulling to my local feature branch (after rebasing the remote feature branch here on GitHub) caused a merge commit :( Trying to see if there's some way I can...

> All your commits will be squashed before merging. Thanks, that's good to know! Having the merge+duplicate commits hanging around was still bugging me though, so I ended up digging...

Glad to see it merged -- thanks for the guidance, @srikanthccv!

Hey folks, I'm new to contributing, and was hoping to give this issue a shot. Skimming through the source code, it looks like solving this issue should be as simple...

Thanks @srikanthccv. I went through all the related PRs and issues to gather context -- if I understand correctly, this is what you're saying needs to be done: In the...

> > LoggingHandler class to take in an additional log_emitter_provider arg > > replace the `log_emitter` arg with provider instance and use it. But `LoggingHandler` class still needs access to...

> You need to obtain the emitter from the provider. It will look something like `self._emitter = get_log_emitter(name, log_emitter_provider=log_emitter_provider)` Ahh gotcha -- thanks!