Skip to content

Commit a896c51

Browse files
stefanbotezStefan Botez
and
Stefan Botez
authored
Include GitHub action on push and pr (aws#1)
Co-authored-by: Stefan Botez <[email protected]>
1 parent df7580c commit a896c51

File tree

4 files changed

+24
-4
lines changed

4 files changed

+24
-4
lines changed
+20
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,20 @@
1+
name: test-on-push-and-pr
2+
3+
on:
4+
push:
5+
branches: [ main ]
6+
pull_request:
7+
branches: [ '*' ]
8+
9+
jobs:
10+
build:
11+
runs-on: ubuntu-latest
12+
13+
steps:
14+
- uses: actions/checkout@v2
15+
- name: Set up node
16+
uses: actions/setup-node@v1
17+
with:
18+
node-version: '12'
19+
- name: Run 'pr' target
20+
run: make pr

Makefile

+1-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,7 @@ dev: init test
4141

4242
# Verifications to run before sending a pull request
4343
.PHONY: pr
44-
pr: dev test-smoke
44+
pr: build dev test-smoke
4545

4646
.PHONY: clean
4747
clean:

scripts/postinstall.sh

+2-2
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,6 @@ if test "${current_path#*$node_modules_path}" != "$current_path" || [ "$BUILD" !
2121

2222
# Clean up source dependencies
2323
rm -rf "$deps_path"/patches
24-
rm -rf "$deps_path"/aws-lambda-cpp*
25-
rm -rf "$deps_path"/curl*
24+
rm -rf "$deps_path"/aws-lambda-cpp*[^gz]$
25+
rm -rf "$deps_path"/curl*[^gz]$
2626
fi

test/integration/codebuild-local/codebuild_build.sh

+1-1
Original file line numberDiff line numberDiff line change
@@ -95,7 +95,7 @@ then
9595
exit 1
9696
fi
9797

98-
docker_command="docker run -it "
98+
docker_command="docker run "
9999
if isOSWindows
100100
then
101101
docker_command+="-v //var/run/docker.sock:/var/run/docker.sock -e "

0 commit comments

Comments
 (0)