Skip to content
This repository has been archived by the owner on Oct 9, 2023. It is now read-only.

Complete github actions migration #156

Merged
merged 11 commits into from
Jun 18, 2020
Merged

Complete github actions migration #156

merged 11 commits into from
Jun 18, 2020

Conversation

EngHabu
Copy link
Contributor

@EngHabu EngHabu commented Jun 15, 2020

TL;DR

Move unit tests, lint and code coverage to Github actions.

Type

  • Bug Fix
  • Feature
  • Plugin

flyteorg/flyte#324

@EngHabu EngHabu requested a review from kumare3 as a code owner June 15, 2020 22:28
@codecov-commenter
Copy link

codecov-commenter commented Jun 15, 2020

Codecov Report

Merging #156 into master will not change coverage.
The diff coverage is n/a.

@EngHabu EngHabu changed the title Add unit tests in github actions Complete github actions migration Jun 15, 2020
@EngHabu EngHabu requested a review from wild-endeavor June 15, 2020 23:55
@kumare3
Copy link
Contributor

kumare3 commented Jun 16, 2020

This is awesome!

kumare3
kumare3 previously approved these changes Jun 16, 2020
wild-endeavor
wild-endeavor previously approved these changes Jun 16, 2020
Copy link
Contributor

@wild-endeavor wild-endeavor left a comment

Choose a reason for hiding this comment

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

can you update flyte as well with the new docker image locatios plz?

@EngHabu EngHabu dismissed stale reviews from wild-endeavor and kumare3 via 33a8631 June 17, 2020 22:52
@EngHabu EngHabu requested a review from wild-endeavor June 17, 2020 22:53
@EngHabu EngHabu merged commit 5553b34 into master Jun 18, 2020
eapolinario pushed a commit to eapolinario/flytepropeller that referenced this pull request Aug 9, 2023
* Add unit tests in github actions

* Remove Travis and update master pipeline

* Push to dockerhub again

* Fix tagging in PRs

* make dockerhub push parallel

* renames

* parallelize image builds

* new line

* Update  boilerplate

* Add reference to how to bump versions
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants