Skip to content

Adding node and relationship filters #98

New issue

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

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

Already on GitHub? # to your account

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chayim
Copy link
Contributor

@chayim chayim commented Jul 21, 2022

unit tests

support for includes regardless of path difference

unit tests

support for includes regardless of path difference
@codecov
Copy link

codecov bot commented Jul 21, 2022

Codecov Report

Merging #98 (58a0a76) into master (c6248f6) will decrease coverage by 1.70%.
The diff coverage is 73.77%.

@@            Coverage Diff             @@
##           master      #98      +/-   ##
==========================================
- Coverage   89.46%   87.75%   -1.71%     
==========================================
  Files           9        9              
  Lines         598      637      +39     
==========================================
+ Hits          535      559      +24     
- Misses         63       78      +15     
Impacted Files Coverage Δ
redisgraph_bulk_loader/config.py 85.00% <50.00%> (-9.12%) ⬇️
redisgraph_bulk_loader/relation_type.py 84.41% <60.00%> (-4.32%) ⬇️
redisgraph_bulk_loader/bulk_insert.py 77.35% <68.75%> (-2.65%) ⬇️
redisgraph_bulk_loader/label.py 88.73% <72.72%> (-3.58%) ⬇️
redisgraph_bulk_loader/entity_file.py 90.35% <90.00%> (-0.15%) ⬇️

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 c6248f6...58a0a76. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Jul 21, 2022

This pull request introduces 5 alerts when merging 58a0a76 into c6248f6 - view on LGTM.com

new alerts:

  • 5 for Except block handles 'BaseException'

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant