Skip to content

Commit a5204d4

Browse files
committed
Removing dispatch-workflows.yml and making CI a bit more simple
1 parent e4f6dde commit a5204d4

2 files changed

Lines changed: 22 additions & 23 deletions

File tree

Lines changed: 22 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,29 @@
1-
name: JavaScript workflow
1+
# Configuration for JavaScript CI
2+
# To force the execution of this workflow, add [run javascript] to your commit message
3+
name: 'CI - JavaScript'
24

35
on:
46
workflow_dispatch:
5-
workflow_call:
7+
68
schedule:
79
- cron: '45 9,21 * * *'
810

11+
push:
12+
branches:
13+
- trunk
14+
pull_request:
15+
branches:
16+
- trunk
17+
918
jobs:
19+
check_workflow:
20+
uses: ./.github/workflows/should-workflow-run.yml
21+
with:
22+
bazel-target-prefix: '//javascript'
23+
1024
node_tests:
25+
if: ${{ needs.check_workflow.outputs.result == 'true' || github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' || contains(toJson(github.event.commits), '[run javascript]') == true }}
26+
needs: check_workflow
1127
runs-on: ubuntu-latest
1228
steps:
1329
- name: Checkout source tree
@@ -47,6 +63,8 @@ jobs:
4763
SELENIUM_BROWSER: firefox
4864

4965
firefox_atom_tests:
66+
if: ${{ needs.check_workflow.outputs.result == 'true' || github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' || contains(toJson(github.event.commits), '[run javascript]') == true }}
67+
needs: check_workflow
5068
runs-on: ubuntu-latest
5169
steps:
5270
- name: Checkout source tree
@@ -83,6 +101,8 @@ jobs:
83101
# As soon as this gets merged https://github.com/facebook/jest/pull/9351, we should upgrade Jest and
84102
# run bazel test javascript/grid-ui:test for these tests
85103
grid_ui_unit_tests:
104+
if: ${{ needs.check_workflow.outputs.result == 'true' || github.event_name == 'schedule' || github.event_name == 'workflow_dispatch' || contains(toJson(github.event.commits), '[run javascript]') == true }}
105+
needs: check_workflow
86106
runs-on: ubuntu-latest
87107
steps:
88108
- name: Checkout source tree

.github/workflows/dispatch-workflows.yml

Lines changed: 0 additions & 21 deletions
This file was deleted.

0 commit comments

Comments
 (0)