Skip to content

Commit 40fcd50

Browse files
dundargocclason
authored andcommitted
revert: "ci: don't run on push"
This reverts commit d1369fc. The reasoning is provided by @justinmk in #122 (comment): It's good to save CI time, but for low-activity repos I think we should err on the side of explicitness. Reasons in favor of CI on push: - to have a baseline, so one can see the history of status checks by looking at master (including squash-merged PRs) - for any (rare) commits that skip the PR process Without this, we don't have a clear concept of "current status", except whatever the last PR was (which also assumes a linear history).
1 parent d1369fc commit 40fcd50

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

.github/workflows/main.yml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,21 @@
11
name: Run tree-sitter tests
22

33
on:
4+
push:
5+
branches: [ master ]
46
pull_request:
7+
branches: [ master ]
58

69
jobs:
710
test:
811
name: Tree-sitter tests
912
runs-on: ubuntu-latest
1013
steps:
1114
- uses: actions/checkout@v4
15+
1216
- uses: actions/setup-node@v4
13-
- run: npm install
14-
- run: npm test
17+
- name: Install Dependencies
18+
run: npm install
19+
20+
- name: Run tests
21+
run: npm test

0 commit comments

Comments
 (0)