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

Update v3.0.0's changelog sentry example #19

Merged
merged 4 commits into from
Jun 10, 2020

Conversation

awichmann-mintel
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Jun 9, 2020

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #19   +/-   ##
=======================================
  Coverage   93.51%   93.52%           
=======================================
  Files          17       18    +1     
  Lines         694      695    +1     
=======================================
+ Hits          649      650    +1     
  Misses         45       45           
Impacted Files Coverage Δ
lpipe/contrib/__init__.py 100.00% <0.00%> (ø)

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 2560ad2...adf5649. Read the comment docs.

CHANGELOG.md Outdated
@@ -12,7 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org).
- Add type hints
- Increase test coverage
- Bugfix: lpipe.sqs.delete_message_batch called boto3 incorrectly
- Breaking: sentry-sdk no longer a required dependency, install with `lpipe = {"extras": ["sentry"], "version": "*"}`, and enable it by setting `process_event(exception_handler=lpipe.contrib.sentry.capture)`
- Breaking: sentry-sdk no longer a required dependency, install with `lpipe = {extras: ["sentry"], version: "*"}`, and enable it by setting `from lpipe.contrib import sentry ... process_event(exception_handler=sentry.capture)`
Copy link
Collaborator

Choose a reason for hiding this comment

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

from lpipe.contrib import sentry; process_event(exception_handler=sentry.capture)

@charlieparkes
Copy link
Collaborator

charlieparkes commented Jun 9, 2020

Improvement in this PR #20

@charlieparkes charlieparkes changed the title add what worked for me Update v3.0.0's changelog sentry example Jun 9, 2020
@charlieparkes
Copy link
Collaborator

#20 fixed this, but feel free to use this PR to fix the extras example

>>> import lpipe
>>> lpipe.contrib.sentry.capture
<function capture at 0x101bb9730>

@charlieparkes charlieparkes merged commit 3932918 into mintel:master Jun 10, 2020
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.

2 participants