Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: change SourceBundleWriter::write_object_with_filter() filter callback to take SourceCode #764

Merged
merged 2 commits into from
Feb 21, 2023

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

None yet

2 participants