ngrx / platform

Reactive State for Angular
https://ngrx.io
Other
7.96k stars 1.95k forks source link

feat(component-store): add migrator for `tapResponse` #4321

Closed rainerhahnekamp closed 2 months ago

rainerhahnekamp commented 2 months ago

Migrates any import of tapResponse from @ngrx/component-store to @ngrx/operators.

I have done this in the same way as the migrator of concatLatestFrom in #4311. So, in the end, it is a better copy & paste.

I could also come up with a generic function that we could reuse for other potential, similar migrations.

Fixes #4261

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] CI related changes
[ ] Documentation content changes
[ ] Other... Please describe:

Closes #4261

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No
netlify[bot] commented 2 months ago

Deploy Preview for ngrx-io canceled.

Built without sensitive environment variables

Name Link
Latest commit aafeb3658a6b1195c5a8f85f88926741931351db
Latest deploy log https://app.netlify.com/sites/ngrx-io/deploys/663bf87e23157c0008007b69