Skip to content

Conversation

@vaind
Copy link
Collaborator

@vaind vaind commented Feb 21, 2023

Part of getsentry/sentry-cli#1482

This enables Sentry-CLI to filter out embedded sources while collecting referenced files.

@vaind vaind requested a review from a team February 21, 2023 10:58
@github-actions
Copy link

github-actions bot commented Feb 21, 2023

Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against 43f4bf4

@vaind vaind force-pushed the feat/bundle-filter-by-type branch from e67ecd2 to 1d093f5 Compare February 21, 2023 11:03
@vaind vaind changed the title feat: expose SourceCode in write_object_with_filter() feat: change SourceBundleWriter::write_object_with_filter() filter callback to take SourceCode Feb 21, 2023
@codecov
Copy link

codecov bot commented Feb 21, 2023

Codecov Report

Merging #764 (015ded6) into master (6394b02) will increase coverage by 0.02%.
The diff coverage is 88.88%.

❗ Current head 015ded6 differs from pull request most recent head 43f4bf4. Consider uploading reports for the commit 43f4bf4 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #764      +/-   ##
==========================================
+ Coverage   74.19%   74.22%   +0.02%     
==========================================
  Files          70       70              
  Lines       15112    15117       +5     
==========================================
+ Hits        11213    11220       +7     
+ Misses       3899     3897       -2     

Co-authored-by: Sebastian Zivota <loewenheim@users.noreply.github.com>
@vaind vaind merged commit 9e55fec into master Feb 21, 2023
@vaind vaind deleted the feat/bundle-filter-by-type branch February 21, 2023 11:24
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