Skip to content
This repository was archived by the owner on Jan 22, 2025. It is now read-only.

Trigger all CI jobs for Dockerfile changes #10651

Merged
ryoqun merged 5 commits into from
Jun 17, 2020
Merged

Trigger all CI jobs for Dockerfile changes #10651

Changes from all commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
47e4eff
Trigger docs job for possible mdbook updates, too
ryoqun Jun 17, 2020
2d51cf9
Update comment
ryoqun Jun 17, 2020
801cf2b
Intentionally trigger ci docs job?
ryoqun Jun 17, 2020
d5fa7c0
Broaden more
ryoqun Jun 17, 2020
aba1c57
Revert "Intentionally trigger ci docs job?"
ryoqun Jun 17, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 4 additions & 2 deletions ci/buildkite-pipeline.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,8 @@ annotate() {
}

# Checks if a CI pull request affects one or more path patterns. Each
# argument is checked in series.
# pattern argument is checked in series. If one of them found to be affected,
# return immediately as such.
#
# Bash regular expressions are permitted in the pattern:
# affects .rs$ -- any file or directory ending in .rs
Expand All @@ -41,7 +42,8 @@ affects() {
# the worse (affected)
return 0
fi
for pattern in "$@"; do
# Assume everyting needs to be tested when any Dockerfile changes
Copy link
Copy Markdown
Contributor Author

@ryoqun ryoqun Jun 17, 2020

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mvines FYI, I think this is safe bet. :)

for pattern in ^ci/docker-rust/Dockerfile ^ci/docker-rust-nightly/Dockerfile "$@"; do
if [[ ${pattern:0:1} = "!" ]]; then
for file in "${affected_files[@]}"; do
if [[ ! $file =~ ${pattern:1} ]]; then
Expand Down