Skip to content

Commit

Permalink
Revert "Superset - Release and Improvements (12-05-2022)"
Browse files Browse the repository at this point in the history
  • Loading branch information
RahulRaj97 authored May 11, 2022
1 parent c4debe6 commit 6d18661
Show file tree
Hide file tree
Showing 108 changed files with 55,120 additions and 50,446 deletions.
24 changes: 2 additions & 22 deletions .github/workflows/bashlib.sh
Original file line number Diff line number Diff line change
Expand Up @@ -183,7 +183,7 @@ cypress-run-all() {
nohup flask run --no-debugger -p $port >"$flasklog" 2>&1 </dev/null &
local flaskProcessId=$!

cypress-run "*/**/!(*.applitools.test.ts)"
cypress-run "*/**/*"

# After job is done, print out Flask log for debugging
say "::group::Flask log for default run"
Expand All @@ -198,7 +198,7 @@ cypress-run-all() {
nohup flask run --no-debugger -p $port >"$flasklog" 2>&1 </dev/null &
local flaskProcessId=$!

cypress-run "sqllab/!(*.applitools.test.ts)" "Backend persist"
cypress-run "sqllab/*" "Backend persist"

# Upload code coverage separately so each page can have separate flags
# -c will clean existing coverage reports, -F means add flags
Expand All @@ -212,23 +212,3 @@ cypress-run-all() {
# make sure the program exits
kill $flaskProcessId
}

cypress-run-applitools() {
local flasklog="${HOME}/flask.log"
local port=8081
export CYPRESS_BASE_URL="http://localhost:${port}"

nohup flask run --no-debugger -p $port >"$flasklog" 2>&1 </dev/null &
local flaskProcessId=$!

cypress-run "*/**/*.applitools.test.ts"

codecov -c -F "cypress" || true

say "::group::Flask log for default run"
cat "$flasklog"
say "::endgroup::"

# make sure the program exits
kill $flaskProcessId
}
113 changes: 0 additions & 113 deletions .github/workflows/superset-applitool-cypress.yml

This file was deleted.

34 changes: 0 additions & 34 deletions .github/workflows/superset-applitools-storybook.yml

This file was deleted.

2 changes: 1 addition & 1 deletion .github/workflows/superset-e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ jobs:
- name: Setup Node.js
uses: actions/setup-node@v2
with:
node-version: "16"
node-version: '16'
- name: Install npm dependencies
if: steps.check.outcome == 'failure'
uses: ./.github/actions/cached-dependencies
Expand Down
Loading

0 comments on commit 6d18661

Please sign in to comment.