Clean up Danger rule for serverSecrets (#5627)
"serverSecrets" is no longer used to access auth in services, so it no longer makes sense to include it in this rule. Ref #3393
This commit is contained in:
@@ -111,10 +111,7 @@ if (allFiles.length > 100) {
|
||||
|
||||
// eslint-disable-next-line promise/prefer-await-to-then
|
||||
danger.git.diffForFile(file).then(({ diff }) => {
|
||||
if (
|
||||
(diff.includes('authHelper') || diff.includes('serverSecrets')) &&
|
||||
!secretsDocs.modified
|
||||
) {
|
||||
if (diff.includes('authHelper') && !secretsDocs.modified) {
|
||||
warn(
|
||||
[
|
||||
`:books: Remember to ensure any changes to \`config.private\` `,
|
||||
|
||||
Reference in New Issue
Block a user