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

[Build] Add Github workflows #9510

Closed
wants to merge 1 commit into from
Closed

Conversation

ktmud
Copy link
Member

@ktmud ktmud commented Apr 12, 2020

Recreating #9490 after some major refactoring.

CATEGORY

  • Build / Development Environment

SUMMARY

Adding GitHub Actions per discussion in #9481 . This is a pure addition to the current build process for now. Nothing changed for Travis CI, except Cypress was upgraded from 3.6.1 to 4.3.0. We could evaluate whether to disable Travis CI once these GitHub Actions become more stable.

Jobs are split into 4 workflows to maximize parallelization:

  • superset-backend.yml: python lint and unit tests
  • superset-frontend.yml: frontend lint and unit tests
  • superset-e2e.yml: end to end tests with Cypress
  • license-check.yml: check license compatibility

A custom GitHub Action was created to generalize the building steps related to caching and dependencies.

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Once successfully integrated, GitHub PRs should have checks like below:

image

Meanwhile, use ktmud#27 to see these checks in action.

TEST PLAN

Make sure CI passes.

ADDITIONAL INFORMATION

N/A

REVIEWERS

@john-bodley @dpgaspar @mistercrunch @craig-rueda

@codecov-io
Copy link

codecov-io commented Apr 12, 2020

Codecov Report

Merging #9510 into master will not change coverage by %.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #9510   +/-   ##
=======================================
  Coverage   58.57%   58.57%           
=======================================
  Files         386      386           
  Lines       12287    12287           
  Branches     3025     3025           
=======================================
  Hits         7197     7197           
  Misses       4906     4906           
  Partials      184      184           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a797465...1e008e9. Read the comment docs.

@ktmud ktmud force-pushed the github-actions branch 2 times, most recently from 1fab574 to b008f58 Compare April 12, 2020 07:41
as replacement to Travis CI.
@nytai
Copy link
Member

nytai commented Apr 12, 2020

@ktmud seems like travis doesn't like your PRs, the hooks don't seem to be registering with github. I think you're the only one experiencing this now 🤔

@nytai
Copy link
Member

nytai commented Apr 12, 2020

Here's the successful build but it didn't register with github.

https://travis-ci.org/github/apache/incubator-superset/builds/673991032

@ktmud ktmud closed this Apr 12, 2020
@ktmud ktmud reopened this Apr 12, 2020
@ktmud
Copy link
Member Author

ktmud commented Apr 13, 2020

Looks like it still won't pass after I close & reopened the PR.

@ktmud
Copy link
Member Author

ktmud commented Apr 13, 2020

Gonna close and recreate.

@ktmud ktmud closed this Apr 13, 2020
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