Skip to content

refactor(deleteActivityProfile): Restructuring#719

Open
ryasmi wants to merge 11 commits intomasterfrom
restructure-experiment
Open

refactor(deleteActivityProfile): Restructuring#719
ryasmi wants to merge 11 commits intomasterfrom
restructure-experiment

Conversation

@ryasmi
Copy link
Copy Markdown
Contributor

@ryasmi ryasmi commented Dec 13, 2020

No description provided.

@lgtm-com
Copy link
Copy Markdown

lgtm-com Bot commented Dec 14, 2020

This pull request fixes 1 alert when merging 7897229 into 22803df - view on LGTM.com

fixed alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link
Copy Markdown

lgtm-com Bot commented Mar 10, 2021

This pull request fixes 1 alert when merging 26d8e30 into 47f7548 - view on LGTM.com

fixed alerts:

  • 1 for Unused variable, import, function or class

@lgtm-com
Copy link
Copy Markdown

lgtm-com Bot commented Mar 10, 2021

This pull request fixes 1 alert when merging a4b0216 into 47f7548 - view on LGTM.com

fixed alerts:

  • 1 for Unused variable, import, function or class

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 10, 2021

Codecov Report

❌ Patch coverage is 71.85629% with 94 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.83%. Comparing base (47f7548) to head (a4b0216).
⚠️ Report is 134 commits behind head on master.

Files with missing lines Patch % Lines
...rofile/utils/getClient/getClientRecordFromMongo.ts 20.00% 40 Missing ⚠️
...eleteActivityProfile/utils/createExpressHandler.ts 58.97% 14 Missing and 2 partials ⚠️
...ns/deleteActivityProfile/utils/translateWarning.ts 43.47% 11 Missing and 2 partials ⚠️
.../deleteActivityProfile/utils/handleErrorViaHttp.ts 71.87% 5 Missing and 4 partials ⚠️
...file/utils/getClient/getClientRecordFromTesters.ts 73.33% 3 Missing and 1 partial ⚠️
...tivityProfile/utils/getAuthConfig/getAuthConfig.ts 66.66% 1 Missing and 1 partial ⚠️
...deleteActivityProfile/utils/getClient/getClient.ts 80.00% 1 Missing and 1 partial ⚠️
...ActivityProfile/utils/getClient/getClientRecord.ts 75.00% 1 Missing and 1 partial ⚠️
...ctivityProfile/utils/getClient/trackClientUsage.ts 71.42% 1 Missing and 1 partial ⚠️
...eteActivityProfile/deleteActivityProfileContent.ts 91.66% 0 Missing and 1 partial ⚠️
... and 3 more
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #719      +/-   ##
==========================================
- Coverage   99.23%   97.83%   -1.40%     
==========================================
  Files         602      631      +29     
  Lines        6235     6559     +324     
  Branches      456      494      +38     
==========================================
+ Hits         6187     6417     +230     
- Misses         11       88      +77     
- Partials       37       54      +17     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant