nest icon indicating copy to clipboard operation
nest copied to clipboard

Published 20 hours ago • verified publisher icon nestjs

refactor(): refactor exception handlers

Open TheKhanj opened this issue 2 years ago • 2 comments

PR Checklist

Please check if your PR fulfills the following requirements:

  • [x] The commit message follows our guidelines: https://github.com/nestjs/nest/blob/master/CONTRIBUTING.md
  • [x] Tests for the changes have been added (for bug fixes / features)
  • [ ] Docs have been added / updated (for bug fixes / features)

PR Type

What kind of change does this PR introduce?

  • [ ] Bugfix
  • [ ] Feature
  • [ ] Code style update (formatting, local variables)
  • [x] 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
  • [x] No

Other information

TheKhanj avatar Aug 18 '22 18:08 TheKhanj

Pull Request Test Coverage Report for Build 45e90302-44ea-46ab-a87d-288a57a77905

  • 13 of 13 (100.0%) changed or added relevant lines in 4 files are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage decreased (-0.004%) to 93.808%
Totals Coverage Status
Change from base Build ec438b34-a56e-4f82-ae11-45373f207a16: -0.004%
Covered Lines: 6090
Relevant Lines: 6492
💛 - Coveralls

coveralls avatar Aug 18 '22 18:08 coveralls

Any opinion about these changes?🤔

TheKhanj avatar Aug 19 '22 12:08 TheKhanj

LGTM

kamilmysliwiec avatar Apr 05 '23 11:04 kamilmysliwiec

Tnx 😍

TheKhanj avatar Apr 05 '23 11:04 TheKhanj