Skip to content

Commit

Permalink
ARROW-8309: [CI] C++/Java/Rust workflows should trigger on changes to…
Browse files Browse the repository at this point in the history
… Flight.proto

This also narrows the scope of the triggers on changes to the `ci/` directory (cc @kszucs )

Closes #6801 from nealrichardson/flight-ci

Authored-by: Neal Richardson <neal.p.richardson@gmail.com>
Signed-off-by: Neal Richardson <neal.p.richardson@gmail.com>
  • Loading branch information
nealrichardson committed Apr 2, 2020
1 parent 5bdb3af commit 67cd34a
Show file tree
Hide file tree
Showing 3 changed files with 18 additions and 4 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/cpp.yml
Original file line number Diff line number Diff line change
Expand Up @@ -23,11 +23,13 @@ on:
- '.github/workflows/cpp.yml'
- 'ci/**'
- 'cpp/**'
- 'format/Flight.proto'
pull_request:
paths:
- '.github/workflows/cpp.yml'
- 'ci/**'
- 'cpp/**'
- 'format/Flight.proto'

env:
ARROW_ENABLE_TIMING_TESTS: OFF
Expand Down
10 changes: 8 additions & 2 deletions .github/workflows/java.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,12 +21,18 @@ on:
push:
paths:
- '.github/workflows/java.yml'
- 'ci/**'
- 'ci/docker/*java*'
- 'ci/scripts/java*.sh'
- 'ci/scripts/util_*.sh'
- 'format/Flight.proto'
- 'java/**'
pull_request:
paths:
- '.github/workflows/java.yml'
- 'ci/**'
- 'ci/docker/*java*'
- 'ci/scripts/java*.sh'
- 'ci/scripts/util_*.sh'
- 'format/Flight.proto'
- 'java/**'

jobs:
Expand Down
10 changes: 8 additions & 2 deletions .github/workflows/rust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -21,13 +21,19 @@ on:
push:
paths:
- '.github/workflows/rust.yml'
- 'ci/**'
- 'ci/docker/*rust*'
- 'ci/scripts/rust_*.sh'
- 'ci/scripts/util_*.sh'
- 'rust/**'
- 'format/Flight.proto'
pull_request:
paths:
- '.github/workflows/rust.yml'
- 'ci/**'
- 'ci/docker/*rust*'
- 'ci/scripts/rust_*.sh'
- 'ci/scripts/util_*.sh'
- 'rust/**'
- 'format/Flight.proto'

jobs:

Expand Down

0 comments on commit 67cd34a

Please sign in to comment.