iamolegga

Results 57 comments of iamolegga

Looks like ci errors are not correlated with current changes. Can someone review this PR, please? 🙂

At the moment it's not possible. Could you describe how you see this feature? What API should be for it?

thanks for the advice, already get rid of the facade in `nestjs-pino`, will consider in current one too

## Coverage report ### Total coverage | Status | Category | Percentage | Covered / Total | | :----: | :--------- | :--------- | :-------------: | | 🟢 | Statements...

Desired behavior is not supported at the moment. You can try any hacks (as presented above) to make it work but on your own risk. This is not tested and...

OK, in next month or two I'm going to investigate if it's possible to make such feature work with different microservices' transports including custom strategies. For implementing this feature in...

Sorry, I'm not a user of pnpm. But rxjs is a dependency of nestjs, and pino-http is a **peer** dependency, that should be installed by end user, and it includes...

So, the solution is to add both `pino` and `rxjs` to peer deps, right? So, rxjs version in peer deps should be the same as used in nestjs's minimal supported...

There are not any tests for `assign` in gql context, should be checked separately. You can try to create a test for this if you want to. I'll get back...

Also I think that it should be configurable, which `level` and maybe something more, like `logArguments`. Good, by the way if anyone is seeing this, PR are welcome, before I...