Skip to content

Conversation

chayim
Copy link
Contributor

@chayim chayim commented Jul 17, 2022

Adding black and other tools to linters. This includes integrating tox-docker, like other projects, and finally moving to GH Actions

@codecov
Copy link

codecov bot commented Jul 17, 2022

Codecov Report

Merging #95 (44068de) into master (a3714c1) will increase coverage by 0.72%.
The diff coverage is 88.99%.

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   88.54%   89.26%   +0.72%     
==========================================
  Files           9        9              
  Lines         576      615      +39     
==========================================
+ Hits          510      549      +39     
  Misses         66       66              
Impacted Files Coverage Δ
redisgraph_bulk_loader/__init__.py 100.00% <ø> (ø)
redisgraph_bulk_loader/relation_type.py 88.73% <78.57%> (+1.40%) ⬆️
redisgraph_bulk_loader/bulk_insert.py 80.35% <87.27%> (+0.54%) ⬆️
redisgraph_bulk_loader/entity_file.py 90.50% <87.30%> (+2.50%) ⬆️
redisgraph_bulk_loader/config.py 94.11% <87.50%> (-5.89%) ⬇️
redisgraph_bulk_loader/query_buffer.py 94.91% <89.65%> (-5.09%) ⬇️
redisgraph_bulk_loader/label.py 92.30% <92.85%> (+0.37%) ⬆️
redisgraph_bulk_loader/bulk_update.py 90.65% <97.14%> (ø)

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 7042ed7...44068de. Read the comment docs.

@RedisGraph RedisGraph deleted a comment from lgtm-com bot Jul 17, 2022
@RedisGraph RedisGraph deleted a comment from lgtm-com bot Jul 17, 2022
@RedisGraph RedisGraph deleted a comment from lgtm-com bot Jul 17, 2022
@RedisGraph RedisGraph deleted a comment from lgtm-com bot Jul 17, 2022
@lgtm-com
Copy link

lgtm-com bot commented Jul 17, 2022

This pull request fixes 2 alerts when merging 44068de into 7042ed7 - view on LGTM.com

fixed alerts:

  • 2 for Except block handles 'BaseException'

@RedisGraph RedisGraph deleted a comment from lgtm-com bot Jul 17, 2022
@RedisGraph RedisGraph deleted a comment from lgtm-com bot Jul 17, 2022
@chayim
Copy link
Contributor Author

chayim commented Jul 17, 2022

Note all GitHub actions now work. The only "red" issue is Circle, which is removed by this PR. The webhook has already been deleted, and merging this, eliminates that problem.

Future PRs will no longer use circle.

@chayim chayim requested a review from AviAvni July 17, 2022 10:21
@chayim chayim changed the title Updating to latest standard Updating to latest standards Jul 17, 2022
@chayim chayim merged commit fc63729 into master Jul 17, 2022
@chayim chayim deleted the ck-baseline branch July 17, 2022 10:43
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