Try to improve Danger rule for PNGs to ignore screenshots (#234)
This commit is contained in:
committed by
GitHub
parent
46c7ede5dc
commit
9b537bcc01
@@ -107,7 +107,9 @@ if (hasChangedViews) {
|
||||
}
|
||||
|
||||
// Check for pngs on resources
|
||||
const hasPngs = editedFiles.filter(file => file.toLowerCase().endsWith(".png")).length > 0
|
||||
const hasPngs = editedFiles.filter(file => {
|
||||
file.toLowerCase().endsWith(".png") && !file.includes("snapshots/images/") // Exclude screenshots
|
||||
}).length > 0
|
||||
if (hasPngs) {
|
||||
warn("You seem to have made changes to some images. Please consider using an vector drawable.")
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user