8000 CI: Tweak Linux-only deployment logic · scijava/scijava-common@055a449 · GitHub
[go: up one dir, main page]
Skip to content
Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly
Appearance settings

Commit 055a449

Browse files
ctruedenctrueden
committed
CI: Tweak Linux-only deployment logic
1 parent 08e86be commit 055a449

File tree

3 files changed

+12
-5
lines changed

3 files changed

+12
-5
lines changed

‎.github/build.sh

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,3 @@
11
#!/bin/sh
22
curl -fsLO https://raw.githubusercontent.com/scijava/scijava-scripts/master/ci-build.sh
3-
# NB: Only the Linux CI node should deploy build artifacts.
4-
NO_DEPLOY=$(test "$(uname)" = Linux || echo 1) sh ci-build.sh
3+
sh ci-build.sh

‎.github/setup.sh

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,10 @@
11
#!/bin/sh
22
curl -fsLO https://raw.githubusercontent.com/scijava/scijava-scripts/master/ci-setup-github-actions.sh
33
sh ci-setup-github-actions.sh
4+
5+
# Let the Linux build handle artifact deployment.
6+
if [ "$(uname)" != Linux ]
7+
then
8+
echo "No deploy -- non-Linux build"
9+
echo "NO_DEPLOY=1" >> $GITHUB_ENV
10+
fi

‎.github/workflows/build.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,14 +1,14 @@
11
name: build
22

33
on:
4-
pull_request:
5-
branches:
6-
- master
74
push:
85
branches:
96
- master
107
tags:
118
- "*-[0-9]+.*"
9+
pull_request:
10+
branches:
11+
- master
1212

1313
jobs:
1414
build:
@@ -28,6 +28,7 @@ jobs:
2828
cache: 'maven'
2929
- name: Set up CI environment
3030
run: .github/setup.sh
31+
shell: bash
3132
- name: Execute the build
3233
run: .github/build.sh
3334
shell: bash

0 commit comments

Comments
 (0)
0