Skip to content

Commit

Permalink
build: add conflict marker check during CI lint
Browse files Browse the repository at this point in the history
PR-URL: #7625
Reviewed-By: Rod Vagg <rod@vagg.org>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
  • Loading branch information
mscdex authored and MylesBorins committed Oct 26, 2016
1 parent cd9e8e0 commit 7766997
Show file tree
Hide file tree
Showing 2 changed files with 9 additions and 1 deletion.
9 changes: 9 additions & 0 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -677,7 +677,16 @@ cpplint:

lint: jslint cpplint

CONFLICT_RE=^>>>>>>> [0-9A-Fa-f]+|^<<<<<<< [A-Za-z]+
lint-ci: lint
@if ! ( grep -IEqrs "$(CONFLICT_RE)" benchmark deps doc lib src test tools ) \
&& ! ( find . -maxdepth 1 -type f | xargs grep -IEqs "$(CONFLICT_RE)" ); then \
exit 0 ; \
else \
echo "" >&2 ; \
echo "Conflict marker detected in one or more files. Please fix them first." >&2 ; \
exit 1 ; \
fi

.PHONY: lint cpplint jslint bench clean docopen docclean doc dist distclean \
check uninstall install install-includes install-bin all staticlib \
Expand Down
1 change: 0 additions & 1 deletion vcbuild.bat
Original file line number Diff line number Diff line change
Expand Up @@ -342,7 +342,6 @@ goto jslint
:jslint
if not defined jslint goto exit
echo running jslint
<<<<<<< d1e2db2a13ddc9fb2f8cd1400b52656910d7374f
%config%\node tools\eslint\bin\eslint.js benchmark lib src test tools --rulesdir tools\eslint-rules
goto exit

Expand Down

0 comments on commit 7766997

Please sign in to comment.