Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: Add TS type for NodeFileTraceReasonType #278

Merged
merged 4 commits into from
Mar 18, 2022

Conversation

styfle
Copy link
Member

@styfle styfle commented Mar 18, 2022

Adds a type to enumerate all possible reasons

@styfle styfle requested a review from ijjk as a code owner March 18, 2022 15:24
@styfle styfle changed the title Add TS type for NodeFileTraceReasonType chore: Add TS type for NodeFileTraceReasonType Mar 18, 2022
@styfle styfle enabled auto-merge (squash) March 18, 2022 15:37
@codecov
Copy link

codecov bot commented Mar 18, 2022

Codecov Report

Merging #278 (dcb353d) into main (ad7e90d) will not change coverage.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main     #278   +/-   ##
=======================================
  Coverage   80.62%   80.62%           
=======================================
  Files          13       13           
  Lines        1461     1461           
  Branches      543      543           
=======================================
  Hits         1178     1178           
  Misses        116      116           
  Partials      167      167           
Impacted Files Coverage Δ
src/node-file-trace.ts 86.57% <100.00%> (ø)

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@styfle styfle merged commit 209b52d into main Mar 18, 2022
@styfle styfle deleted the add-node-file-reason-type branch March 18, 2022 19:07
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.

2 participants