-
Notifications
You must be signed in to change notification settings - Fork 171
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
Reopen unit test in CI #529
Comments
@Xunzhuo Hi, can I assign it to you? |
Sure, this is what I want too, since the new pipelines are running, there are a lot of things need to do make it stable. |
I will start a new issue to cover all the tasks related to the new Pipelines, make it stable. |
ok! |
Should not we test cases in |
cc @seeflood, please check this if it is missed:) |
@Xunzhuo We should. I didn't put it here because your scripts already test this package |
Our CI won't run unit tests due to #483 . Now it's time to reopen it.
We shall:
Reopen unit test in CI
layotto/.github/workflows/layotto-ci.yml
Line 59 in 223945e
Don't test code under
./cmd/...
packagelayotto/make/golang.mk
Line 87 in 223945e
Test code under
sdk/go-sdk
packageTest code under
component
package after Unit tests fail incomponent
package; 修复ut报错 #528 fixedThe text was updated successfully, but these errors were encountered: