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

fix: fix ProfileApi url check #1893

Merged
merged 1 commit into from
May 28, 2022
Merged

fix: fix ProfileApi url check #1893

merged 1 commit into from
May 28, 2022

Conversation

corgiboygsj
Copy link
Member

@corgiboygsj corgiboygsj commented May 27, 2022

Delete blank url check, because ProfileApi has api in use "/"

@codecov
Copy link

codecov bot commented May 27, 2022

Codecov Report

Merging #1893 (af32ace) into master (88d71e7) will increase coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             master    #1893    +/-   ##
==========================================
  Coverage     66.75%   66.75%            
- Complexity      728      980   +252     
==========================================
  Files           451      451            
  Lines         38207    38206     -1     
  Branches       5407     5407            
==========================================
+ Hits          25505    25506     +1     
+ Misses        10185    10183     -2     
  Partials       2517     2517            
Impacted Files Coverage Δ
...idu/hugegraph/api/filter/AuthenticationFilter.java 55.00% <ø> (-0.45%) ⬇️
...egraph/backend/store/cassandra/CassandraShard.java 54.54% <0.00%> (+1.81%) ⬆️

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 88d71e7...af32ace. Read the comment docs.

Copy link
Contributor

@wangyao2016 wangyao2016 left a comment

Choose a reason for hiding this comment

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

LGTM

@imbajin imbajin changed the title fix bug: fix ProfileApi url check fix: fix ProfileApi url check May 28, 2022
@imbajin imbajin merged commit 1118418 into master May 28, 2022
@imbajin imbajin deleted the fix-api-url-check branch May 28, 2022 05:52
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.

4 participants