Skip to content

Commit 3e90169

Browse files
committed
test: Fix syntax error, add php -l to make lint
Follows-up f722dd7. This file is only run by the periodic "smoke" job in CI, and so didn't get noticed until a few minutes after deploying the change. Not a big deal since the file is indeed only used by that job, and not anywhere in prod. But.. getting that feedback earlier when there is an obvious statically detectable mistake will be useful so let's lint PHP files in `make lint` both locally and in CI for commit/PR.
1 parent f722dd7 commit 3e90169

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

Makefile

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ doc/wordpress.md:
2626
lint:
2727
puppet-lint --fail-on-warnings .
2828
@ ./bin/build_wordpress_md.sh --verify
29+
@ php -l test/*.php
2930

3031
test: test-codeorigin-prod-http test-codeorigin-prod-https test-codeorigin-next-http test-codeorigin-next-https test-codeorigin-stage test-contentorigin-prod test-miscweb test-wpdocs test-releases
3132

test/WpdocsTest.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,7 +29,7 @@
2929
'https://jquerymobile.com/blog/feed' => 'https://blog.jquerymobile.com/feed',
3030
'https://jquerymobile.com/blog/feed/' => 'https://blog.jquerymobile.com/feed/',
3131
'https://jquerymobile.com/blog/2011/11/16/announcing-jquery-mobile-1-0/' => 'https://blog.jquerymobile.com/2011/11/16/announcing-jquery-mobile-1-0/',
32-
'https://jquerymobile.com/demos/1.3.0-beta.1/docs/demos/panels/panel-nav-form.html' => 'https://demos.jquerymobile.com/1.3.0-beta.1/docs/demos/panels/panel-nav-form.html'
32+
'https://jquerymobile.com/demos/1.3.0-beta.1/docs/demos/panels/panel-nav-form.html' => 'https://demos.jquerymobile.com/1.3.0-beta.1/docs/demos/panels/panel-nav-form.html',
3333

3434
'https://jquery.org/feed/' => 'https://meetings.jquery.org/feed/',
3535

0 commit comments

Comments
 (0)