Skip to content
This repository has been archived by the owner on Aug 30, 2023. It is now read-only.

boot time should be UTC #309

Merged
merged 3 commits into from
Mar 20, 2020
Merged

boot time should be UTC #309

merged 3 commits into from
Mar 20, 2020

Conversation

marandaneto
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

boot time should be UTC

💡 Motivation and Context

all dates are UTC but boot time

💚 How did you test it?

running it.

📝 Checklist

  • I reviewed submitted code
  • I added tests to verify changes
  • All tests passing

🔮 Next steps

@codecov-io
Copy link

codecov-io commented Mar 20, 2020

Codecov Report

Merging #309 into master will decrease coverage by <.01%.
The diff coverage is 0%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #309      +/-   ##
============================================
- Coverage     57.49%   57.48%   -0.01%     
- Complexity      587      588       +1     
============================================
  Files            73       73              
  Lines          2790     2792       +2     
  Branches        236      236              
============================================
+ Hits           1604     1605       +1     
- Misses         1069     1071       +2     
+ Partials        117      116       -1
Impacted Files Coverage Δ Complexity Δ
...y-core/src/main/java/io/sentry/core/DateUtils.java 50% <0%> (-5.56%) 4 <0> (ø)
...c/main/java/io/sentry/core/DirectoryProcessor.java 80% <0%> (+4%) 7% <0%> (+1%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8a3d35e...96c3a49. Read the comment docs.

@marandaneto marandaneto merged commit 2d02504 into master Mar 20, 2020
@marandaneto marandaneto deleted the fix/boottime-utc branch March 20, 2020 19:19
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants