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.

Sign up for GitHub

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

Jump to bottom

Add build and test GitHub action #66

Merged
merged 4 commits into from
Apr 12, 2021
Merged

Add build and test GitHub action #66

merged 4 commits into from
Apr 12, 2021

Conversation

martinkennelly
Copy link
Member

@martinkennelly martinkennelly commented Feb 17, 2021 •
edited
Loading

Part of issue #68

@martinkennelly martinkennelly changed the title Add build and test github action Add build and test GitHub action Feb 17, 2021
@martinkennelly
Copy link
Member Author

I was thinking - you can also add cron job to these actions - do we want to add this to this PR? Twice a week ?

@martinkennelly
Copy link
Member Author

Added support for test and build for multiple golang versions.
#74 needs to be merged before this GA executes correctly.

@zshi-redhat
Copy link
Collaborator

Added support for test and build for multiple golang versions.
#74 needs to be merged before this GA executes correctly.

#74 was merged, shall we re-run the workflow?

@martinkennelly
Add build and test github action
b2ba620
@martinkennelly
Add cron job of build and test GA
7a8a90c 
Add scheduled execution of build and test daily at 10 am.

Signed-off-by: Kennelly, Martin <[email protected]>
@martinkennelly
Remove conditional execution of build & test
9914b83 
There is no need to gate build and test with a condition.

Signed-off-by: Kennelly, Martin <[email protected]>
@martinkennelly
Test for multiple golang versions
20519c9 
Signed-off-by: Kennelly, Martin <[email protected]>
@martinkennelly
Copy link
Member Author

@zshi-redhat rebased against master to trigger workflow tests.

@zshi-redhat zshi-redhat merged commit ab7bd3e into k8snetworkplumbingwg:master Apr 12, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zshi-redhat zshi-redhat zshi-redhat approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@martinkennelly @zshi-redhat