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 install and test action (Windows) #49

Merged
merged 3 commits into from
Mar 26, 2021
Merged

Conversation

Yikun
Copy link
Contributor

@Yikun Yikun commented Mar 26, 2021

What type of PR is this?

/kind enhancement

What does this PR do / why do we need it:

The install and test github action has been enabled in this patch, the first version of the action test is only runs on windows-latest node, that means it is only tested in windows server 2019 env.

The install and test github action has been enabled in this patch, the first version of the action test is onlt runs on windows-latest node, that means it only is tested in windows server 2019 env.
@CLAassistant
Copy link

CLAassistant commented Mar 26, 2021

CLA assistant check
All committers have signed the CLA.

@leonwanghui
Copy link
Contributor

@Yikun Thanks a lot!

@Yikun
Copy link
Contributor Author

Yikun commented Mar 26, 2021

cc @leonwanghui

This patch only adds the windows test, as I mentioned before, the Travis CI changed the trial (free) plan with a 10K credit allotment.

So looks like the github action is a better choice, and I'd like to move our tinyms CI from travis CI to action completely in another PR.

@leonwanghui
Copy link
Contributor

cc @leonwanghui

This patch only adds the windows test, as I mentioned before, the Travis CI changed the trial (free) plan with a 10K credit allotment.

So looks like the github action is the better choice, and I'd like to move our tinyms CI from travis CI to action completely in another PR.

Sure, that would be great!

@leonwanghui leonwanghui merged commit 967e372 into tinyms-ai:main Mar 26, 2021
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.

3 participants