Skip to content

Commit

Permalink
Adjust validator .js paths in whitelist to allow me to merge PR 4306. (
Browse files Browse the repository at this point in the history
…ampproject#4318)

Unfortunately I didn't include the 'document' whitelist in my previous attempt (ampproject#4308) so the presubmit still fails.
  • Loading branch information
powdercloud authored and ariangibson committed Sep 7, 2016
1 parent 82847ae commit 06e599b
Showing 1 changed file with 8 additions and 6 deletions.
14 changes: 8 additions & 6 deletions build-system/tasks/presubmit-checks.js
Expand Up @@ -354,10 +354,11 @@ var forbiddenTerms = {
'\\.startsWith': {
message: es6polyfill,
whitelist: [
'validator/index.js', // NodeJs only.
'validator/tokenize-css.js',
'validator/validator-full.js',
'validator/validator.js',
'validator/tokenize-css.js', // TODO(powdercloud): remove
'validator/validator-full.js', // TODO(powdercloud): remove
'validator/validator.js', // TODO(powdercloud): remove
'validator/engine/tokenize-css.js',
'validator/engine/validator.js',
// exports.startsWith occurs in babel generated code.
'dist.3p/current/integration.js',
]
Expand Down Expand Up @@ -397,8 +398,9 @@ var forbiddenTerms = {
'testing/iframe.js',
'testing/screenshots/make-screenshot.js',
'tools/experiments/experiments.js',
'validator/validator-full.js',
'validator/validator.js',
'validator/validator-full.js', // TODO(powdercloud): remove
'validator/validator.js', // TODO(powdercloud): remove
'validator/engine/validator.js',
],
},
'getUnconfirmedReferrerUrl': {
Expand Down

0 comments on commit 06e599b

Please sign in to comment.