Skip to content

fix(microservices): if indexOf return 0 will if will be falsy#16401

Merged
kamilmysliwiec merged 1 commit into
nestjs:masterfrom
cuiweixie:bugfix
Feb 20, 2026
Merged

fix(microservices): if indexOf return 0 will if will be falsy#16401
kamilmysliwiec merged 1 commit into
nestjs:masterfrom
cuiweixie:bugfix

Conversation

@cuiweixie

@cuiweixie cuiweixie commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

PR Checklist

Please check if your PR fulfills the following requirements:

PR Type

What kind of change does this PR introduce?

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

What is the current behavior?

Issue Number: N/A

What is the new behavior?

Does this PR introduce a breaking change?

  • Yes
  • No

Other information

@cuiweixie cuiweixie changed the title fix: if indexOf return 0 will if will be falsy fix(microservices): if indexOf return 0 will if will be falsy Feb 20, 2026
@coveralls

Copy link
Copy Markdown

Pull Request Test Coverage Report for Build 7af47d75-6fd7-4b4c-8f4a-e46a58848071

Details

  • 1 of 1 (100.0%) changed or added relevant line in 1 file are covered.
  • 2 unchanged lines in 1 file lost coverage.
  • Overall coverage increased (+0.02%) to 89.853%

Files with Coverage Reduction New Missed Lines %
packages/microservices/server/server-grpc.ts 2 86.14%
Totals Coverage Status
Change from base Build b8992e80-6d53-42f7-b7cf-bd59676cd4c1: 0.02%
Covered Lines: 7465
Relevant Lines: 8308

💛 - Coveralls

@kamilmysliwiec

Copy link
Copy Markdown
Member

AFAIR it should never be just "cancelled" that's why "indexOf" won't be 0 anyway. Doesn't hurt to merge it anyways

@kamilmysliwiec kamilmysliwiec merged commit f39b8cf into nestjs:master Feb 20, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants