Skip to content
GitLab
Explore
Sign in
Register
Primary navigation
Search or go to…
Project
G
gitlab-ci
Manage
Activity
Members
Code
Repository
Branches
Commits
Tags
Repository graph
Compare revisions
Help
Help
Support
GitLab documentation
Compare GitLab plans
GitLab community forum
Contribute to GitLab
Provide feedback
Keyboard shortcuts
?
Snippets
Groups
Projects
Show more breadcrumbs
wpdesk
gitlab-ci
Commits
f52dde36
Verified
Commit
f52dde36
authored
11 months ago
by
Bartek Jaskulski
Browse files
Options
Downloads
Patches
Plain Diff
fix: further improvements to lint job
Signed-off-by:
Bart Jaskulski
<
bjaskulski@protonmail.com
>
parent
fabad4f6
No related branches found
No related tags found
No related merge requests found
Changes
1
Show whitespace changes
Inline
Side-by-side
Showing
1 changed file
includes/tests/lint.yml
+9
-3
9 additions, 3 deletions
includes/tests/lint.yml
with
9 additions
and
3 deletions
includes/tests/lint.yml
+
9
−
3
View file @
f52dde36
...
...
@@ -7,9 +7,15 @@
-
composer install --no-dev
-
rm auth.json
script
:
-
LINT=$(find . -name '*.php' -print0 |xargs -n1 -0 -P16 php -l)
-
RESULT="$?"
-
if [[ "$RESULT" -ne 0 ]]; then echo -e "$LINT" | grep -v 'No syntax errors'; fi
-
|
LINT="$(find . -name '*.php' -print0 |xargs -n1 -0 -P16 php -l 2>&1)"; \
RESULT="$?"; \
if [[ "$RESULT" -ne 0 ]]; then
echo -e "$LINT" | grep -v 'No syntax errors';
exit 0;
else
exit "$RESULT"
fi
lint:8.2
:
extends
:
.lint
...
...
This diff is collapsed.
Click to expand it.
Preview
0%
Loading
Try again
or
attach a new file
.
Cancel
You are about to add
0
people
to the discussion. Proceed with caution.
Finish editing this message first!
Save comment
Cancel
Please
register
or
sign in
to comment