Skip to content

fix: Skip deno stack trace from error message#787

Merged
Natoandro merged 2 commits into
mainfrom
met-568-avoid-dirty-panic
Jul 9, 2024
Merged

fix: Skip deno stack trace from error message#787
Natoandro merged 2 commits into
mainfrom
met-568-avoid-dirty-panic

Conversation

@Natoandro

Copy link
Copy Markdown
Contributor

Skip the deno stack trace from the error message when tg_manage fails.

Migration notes

N/A

  • The change comes with new or modified tests
  • Hard-to-understand functions have explanatory comments
  • End-user documentation is updated to reflect the change

@linear

linear Bot commented Jul 9, 2024

Copy link
Copy Markdown

@codecov

codecov Bot commented Jul 9, 2024

Copy link
Copy Markdown

Codecov Report

Attention: Patch coverage is 0% with 41 lines in your changes missing coverage. Please review.

Project coverage is 69.91%. Comparing base (f367daf) to head (30e9b0f).

Files Patch % Lines
typegraph/deno/sdk/src/tg_manage.ts 0.00% 41 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #787   +/-   ##
=======================================
  Coverage   69.91%   69.91%           
=======================================
  Files         139      139           
  Lines       16181    16181           
  Branches     1475     1475           
=======================================
  Hits        11313    11313           
  Misses       4843     4843           
  Partials       25       25           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@destifo

destifo commented Jul 9, 2024

Copy link
Copy Markdown
Contributor

we should really add the strict deno fmt though 😅

@Natoandro Natoandro merged commit a2a97ed into main Jul 9, 2024
@Natoandro Natoandro deleted the met-568-avoid-dirty-panic branch July 9, 2024 11:17
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