chiark
/
gitweb
/
~ianmdlvl
/
fdroidserver.git
/ commitdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
| commitdiff |
tree
raw
|
patch
|
inline
| side by side (parent:
550d907
)
lint: be more strict about redundant summaries
author
Daniel Martí
<mvdan@mvdan.cc>
Fri, 22 May 2015 07:33:15 +0000
(09:33 +0200)
committer
Daniel Martí
<mvdan@mvdan.cc>
Fri, 22 May 2015 07:33:15 +0000
(09:33 +0200)
fdroidserver/lint.py
patch
|
blob
|
history
diff --git
a/fdroidserver/lint.py
b/fdroidserver/lint.py
index 8b9139b544d4002be59b84c40a2715ccc1846e9f..55722506e7acd7077a450120fac3f33431fbc7e5 100644
(file)
--- a/
fdroidserver/lint.py
+++ b/
fdroidserver/lint.py
@@
-81,6
+81,10
@@
regex_warnings = {
'Summary': [
(re.compile(r'^$'),
"Summary yet to be filled"),
'Summary': [
(re.compile(r'^$'),
"Summary yet to be filled"),
+ (re.compile(r'.*\b(free software|open source)\b.*', re.IGNORECASE),
+ "No need to specify that the app is Free Software"),
+ (re.compile(r'.*((your|for).*android|android.*(app|device|client|port|version))', re.IGNORECASE),
+ "No need to specify that the app is for Android"),
],
'Description': [
(re.compile(r'^No description available$'),
],
'Description': [
(re.compile(r'^No description available$'),
@@
-114,8
+118,6
@@
regex_pedantic = {
"/issues is often enough on its own"),
],
'Summary': [
"/issues is often enough on its own"),
],
'Summary': [
- (re.compile(r'.*\b(free software|open source)\b.*', re.IGNORECASE),
- "No need to specify that the app is Free Software"),
(re.compile(r'.*[a-z0-9][.!?][ $]'),
"Punctuation should be avoided"),
],
(re.compile(r'.*[a-z0-9][.!?][ $]'),
"Punctuation should be avoided"),
],