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

没有查询权限时,增加申请链接 #904

Merged
merged 5 commits into from
Oct 14, 2020
Merged

Conversation

dongqianzheng
Copy link
Contributor

@dongqianzheng dongqianzheng commented Oct 9, 2020

image

@request-info
Copy link

request-info bot commented Oct 9, 2020

你好!感谢你反馈的问题/bug,但是你的描述好像是空的,我们需要你完整的信息,这样才能帮你解决问题 如果不知道怎么写,在新建issue的时候有若干个模板可供选择,祝好!

Copy link
Owner

@hhyo hhyo left a comment

Choose a reason for hiding this comment

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

按照ci结果调整一下测试用例

@codecov
Copy link

codecov bot commented Oct 14, 2020

Codecov Report

Merging #904 into master will not change coverage.
The diff coverage is 75.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #904   +/-   ##
=======================================
  Coverage   77.31%   77.31%           
=======================================
  Files          76       76           
  Lines       11503    11503           
=======================================
  Hits         8893     8893           
  Misses       2610     2610           
Impacted Files Coverage Δ
sql/query.py 70.74% <0.00%> (ø)
sql/query_privileges.py 56.92% <100.00%> (ø)
sql/tests.py 100.00% <100.00%> (ø)

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 3343a70...98bce19. Read the comment docs.

@LeoQuote
Copy link
Collaborator

麻烦 rebase 下master吧,master 有更新

@dongqianzheng
Copy link
Contributor Author

我要怎么操作呢?先把我的仓库删掉,然后再fork一个最新的,再修改,提pr吗?

@hhyo hhyo merged commit 67f6afa into hhyo:master Oct 14, 2020
hhyo pushed a commit that referenced this pull request Oct 31, 2020
* 没有查询权限时,增加申请链接
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants