Skip to content

Conversation

saartochner-lumigo
Copy link
Contributor

No description provided.

saartochner-lumigo and others added 30 commits October 4, 2020 17:13
…acer into RD-3690-redis-sqlalchemy-elasticsearch
Copy link
Contributor

@nirLumigo nirLumigo left a comment

Choose a reason for hiding this comment

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

Good job :)

Comment on lines +68 to +71
with lumigo_safe_execute("sqlalchemy: listen to engine"):
listen(result, "before_cursor_execute", _before_cursor_execute)
listen(result, "after_cursor_execute", _after_cursor_execute)
listen(result, "handle_error", _handle_error)
Copy link
Contributor

Choose a reason for hiding this comment

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

listen might be None

Copy link
Contributor

Choose a reason for hiding this comment

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

I think you forgot to commit this change 😅

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I did! checkout line 80

@codecov
Copy link

codecov bot commented Oct 25, 2020

Codecov Report

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

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #159      +/-   ##
==========================================
+ Coverage   96.21%   97.99%   +1.77%     
==========================================
  Files          11       22      +11     
  Lines         952     1597     +645     
==========================================
+ Hits          916     1565     +649     
+ Misses         36       32       -4     
Impacted Files Coverage Δ
lumigo_tracer/sync_http/handler.py
lumigo_tracer/sync_http/sync_hook.py
lumigo_tracer/parsers/utils.py
lumigo_tracer/parsers/parser.py
lumigo_tracer/utils.py
lumigo_tracer/parsers/http_data_classes.py
lumigo_tracer/parsers/event_parser.py
lumigo_tracer/wrappers/http/http_data_classes.py 100.00% <0.00%> (ø)
lumigo_tracer/event/event_trigger.py 100.00% <0.00%> (ø)
lumigo_tracer/wrappers/redis/redis_wrapper.py 96.36% <0.00%> (ø)
... and 16 more

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 98e7170...c9d82e5. Read the comment docs.

@saartochner-lumigo saartochner-lumigo merged commit b0ff649 into master Oct 25, 2020
@saartochner-lumigo saartochner-lumigo deleted the RD-3763-sqlalchemy branch October 25, 2020 09:05
saartochner-lumigo pushed a commit that referenced this pull request Oct 25, 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