Skip to content

Commit e3771f0

Browse files
author
Erwin Dondorp
committed
replaceAll is not allowed due to primitive regression test
1 parent 0cbfcf2 commit e3771f0

1 file changed

Lines changed: 9 additions & 0 deletions

File tree

.github/workflows/node.js.yml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,3 +37,12 @@ jobs:
3737
- run: npm run eslint
3838

3939
- run: npm run test:coverage
40+
41+
- run: |
42+
echo "--- searching for 'replaceAll' ---"
43+
CNT=`find saltgui/static/scripts -name "*.js" | xargs fgrep "replaceAll" | wc -l`
44+
if [ $CNT != 0 ]; then
45+
find saltgui/static/scripts -name "*.js" | xargs fgrep "replaceAll"
46+
echo "'replaceAll' must be replaced with 'replace(//g, ...)' due to regression test"
47+
exit 1
48+
fi

0 commit comments

Comments
 (0)