diff options
author | bors-servo <lbergstrom+bors@mozilla.com> | 2016-05-19 07:25:12 -0700 |
---|---|---|
committer | bors-servo <lbergstrom+bors@mozilla.com> | 2016-05-19 07:25:12 -0700 |
commit | 2d5dc8fa6db4d714c2018d78f502b3298a32cfaf (patch) | |
tree | 20bb4109d8c1bda54531f7f840429b4cc90bd8df | |
parent | be9dfb2786c67d92e34b365b32935ed2342bb691 (diff) | |
parent | a6990b26acf689e4c81cdb31b12133867425c0d1 (diff) | |
download | servo-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-x | etc/ci/check_no_unwrap.sh | 6 |
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[@]}" |