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

refactor: replace the go model from custom to the official #439

Merged
merged 5 commits into from
May 18, 2024

Conversation

dshyjtdes8888
Copy link
Contributor

refactor: Replace the go model from custom to the official (#143)
close #143

@CLAassistant
Copy link

CLAassistant commented May 17, 2024

CLA assistant check
All committers have signed the CLA.

@LinuxSuRen LinuxSuRen changed the title refactor: Replace the go model from custom to the official (#143) refactor: replace the go model from custom to the official (#143) May 17, 2024
@LinuxSuRen LinuxSuRen changed the title refactor: replace the go model from custom to the official (#143) refactor: replace the go model from custom to the official May 17, 2024
Copy link
Owner

@LinuxSuRen LinuxSuRen left a comment

Choose a reason for hiding this comment

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

The unit tests failed. See also

image

@LinuxSuRen LinuxSuRen added the ospp 开源之夏 https://summer-ospp.ac.cn/ label May 17, 2024
Copy link

sonarcloud bot commented May 17, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@dshyjtdes8888
Copy link
Contributor Author

您好,这个Qodana的报错我不是很懂,我查了一下只有这个讨论中有提到这个问题,里面有提到需要修改配置,但我不确定这样做会不会带来问题:JetBrains/Qodana#209

@LinuxSuRen
Copy link
Owner

您可以忽略 Qodana 这个错误,是由于 GitHub Actions 的权限机制导致的。这个 PR 我抽空再测试下哈,您可以再等等。

Copy link
Owner

@LinuxSuRen LinuxSuRen left a comment

Choose a reason for hiding this comment

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

LGTM

And welcome to be the 22nd contributor.

@LinuxSuRen LinuxSuRen merged commit 1dd9dd4 into LinuxSuRen:master May 18, 2024
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore ospp 开源之夏 https://summer-ospp.ac.cn/
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Replace the go model from custom to the official
3 participants