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

Add deprecated decorator and apply it on gremlin #3007

Merged
merged 2 commits into from
Jul 12, 2023

Conversation

siyuan0322
Copy link
Collaborator

No description provided.

sighingnow
sighingnow previously approved these changes Jul 12, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 12, 2023

Codecov Report

Merging #3007 (17cc4bf) into main (9bf63ac) will decrease coverage by 0.61%.
The diff coverage is 93.33%.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main    #3007      +/-   ##
==========================================
- Coverage   73.91%   73.30%   -0.61%     
==========================================
  Files          99       99              
  Lines       10698    10693       -5     
==========================================
- Hits         7907     7839      -68     
- Misses       2791     2854      +63     
Impacted Files Coverage Δ
python/graphscope/client/connection.py 38.31% <0.00%> (ø)
python/graphscope/client/session.py 75.97% <100.00%> (-1.19%) ⬇️
python/graphscope/framework/utils.py 68.18% <100.00%> (+0.62%) ⬆️
python/graphscope/interactive/query.py 83.33% <100.00%> (+1.04%) ⬆️

... and 9 files with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a9cb7f8...17cc4bf. Read the comment docs.

@siyuan0322 siyuan0322 merged commit ce0407a into alibaba:main Jul 12, 2023
28 checks passed
@siyuan0322 siyuan0322 deleted the zsy/deprecated branch July 12, 2023 05:30
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

3 participants