Skip to content

fix installation id catches dir not exist#1159

Merged
bruno-garcia merged 2 commits intomainfrom
fix/session-dir-not-exist
Aug 17, 2021
Merged

fix installation id catches dir not exist#1159
bruno-garcia merged 2 commits intomainfrom
fix/session-dir-not-exist

Conversation

@bruno-garcia
Copy link
Member

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Aug 17, 2021

Codecov Report

❌ Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.47%. Comparing base (fb2372a) to head (21b5add).
⚠️ Report is 2157 commits behind head on main.

Files with missing lines Patch % Lines
src/Sentry/GlobalSessionManager.cs 0.00% 6 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1159      +/-   ##
==========================================
- Coverage   80.56%   80.47%   -0.09%     
==========================================
  Files         204      204              
  Lines        6683     6689       +6     
  Branches     1481     1482       +1     
==========================================
- Hits         5384     5383       -1     
- Misses        817      823       +6     
- Partials      482      483       +1     

☔ 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.

@bruno-garcia bruno-garcia merged commit 45f95bb into main Aug 17, 2021
@bruno-garcia bruno-garcia deleted the fix/session-dir-not-exist branch August 17, 2021 16:46

### Fixes

- Installation id catches dir not exist([#1159](https://github.com/getsentry/sentry-dotnet/pull/1159))
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Installation id catches dir not exist (#1159)

We need to fix the missing space from Dange :)

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