aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorbors-servo <lbergstrom+bors@mozilla.com>2016-05-19 07:25:12 -0700
committerbors-servo <lbergstrom+bors@mozilla.com>2016-05-19 07:25:12 -0700
commit2d5dc8fa6db4d714c2018d78f502b3298a32cfaf (patch)
tree20bb4109d8c1bda54531f7f840429b4cc90bd8df
parentbe9dfb2786c67d92e34b365b32935ed2342bb691 (diff)
parenta6990b26acf689e4c81cdb31b12133867425c0d1 (diff)
downloadservo-2d5dc8fa6db4d714c2018d78f502b3298a32cfaf.tar.gz
servo-2d5dc8fa6db4d714c2018d78f502b3298a32cfaf.zip
Auto merge of #11251 - asajeffrey:check-no-unwrap-check-for-files, r=aneeshusa
Added test that files exist to etc/ci/check_no_unwrap.sh. Thank you for contributing to Servo! Please replace each `[ ]` by `[X]` when the step is complete, and replace `__` with appropriate data: - [X] `./mach build -d` does not report any errors - [X] `./mach test-tidy --faster` does not report any errors - [X] No github issue. Either: - [ ] There are tests for these changes OR - [X] These changes do not require tests because this PR updates test code. Pull requests that do not address these steps are welcome, but they will require additional verification as part of the review process. <!-- Reviewable:start --> --- This change is [<img src="https://reviewable.io/review_button.svg" height="35" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/servo/11251) <!-- Reviewable:end -->
-rwxr-xr-xetc/ci/check_no_unwrap.sh6
1 files changed, 5 insertions, 1 deletions
diff --git a/etc/ci/check_no_unwrap.sh b/etc/ci/check_no_unwrap.sh
index e70fa31192a..2d949cd591c 100755
--- a/etc/ci/check_no_unwrap.sh
+++ b/etc/ci/check_no_unwrap.sh
@@ -13,4 +13,8 @@ FILES=("components/compositing/compositor.rs"
"components/compositing/pipeline.rs"
"components/constellation/constellation.rs")
-! grep -n "unwrap(\|panic!(" "${FILES[@]}"
+# make sure the files exist
+ls -1 "${FILES[@]}"
+
+# make sure the files do not contain "unwrap" or "panic!"
+! grep --line-number --with-filename "unwrap(\|panic!(" "${FILES[@]}"