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

listing assets file on the wrong dir #156

Merged
merged 1 commit into from
Nov 18, 2019

Conversation

marandaneto
Copy link
Contributor

📢 Type of change

  • Bugfix
  • New feature
  • Enhancement
  • Refactoring

📜 Description

listing assets file on the wrong dir (when looking up for sentry-debug-meta.properties)

💡 Motivation and Context

sentry-debug-meta.properties was not being found even if it was there.

💚 How did you test it?

debugging with proguard enabled

📝 Checklist

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

🔮 Next steps

@codecov-io
Copy link

Codecov Report

Merging #156 into master will increase coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##             master    #156      +/-   ##
===========================================
+ Coverage     57.06%   57.1%   +0.04%     
- Complexity      515     516       +1     
===========================================
  Files            72      72              
  Lines          2492    2492              
  Branches        219     219              
===========================================
+ Hits           1422    1423       +1     
  Misses          962     962              
+ Partials        108     107       -1
Impacted Files Coverage Δ Complexity Δ
...c/main/java/io/sentry/core/DirectoryProcessor.java 74.07% <0%> (+3.7%) 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 22d8b4e...9ded1cd. Read the comment docs.

@bruno-garcia bruno-garcia merged commit 4672bac into master Nov 18, 2019
@bruno-garcia bruno-garcia deleted the bug/sentry_debug_metadata_dir branch November 18, 2019 20:40
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.

3 participants