mirror of
https://gitlab.com/fdroid/fdroidserver.git
synced 2024-10-05 18:50:09 +02:00
lint: get rid of -p/--pedantic
Only one check was left, and it's of little use anyway.
This commit is contained in:
parent
5f894f1d38
commit
d0a10e05d9
@ -197,8 +197,8 @@ __complete_rewritemeta() {
|
|||||||
}
|
}
|
||||||
|
|
||||||
__complete_lint() {
|
__complete_lint() {
|
||||||
opts="-v -q -p"
|
opts="-v -q"
|
||||||
lopts="--verbose --quiet --pedantic"
|
lopts="--verbose --quiet"
|
||||||
case "${cur}" in
|
case "${cur}" in
|
||||||
-*)
|
-*)
|
||||||
__complete_options
|
__complete_options
|
||||||
|
@ -106,18 +106,12 @@ def main():
|
|||||||
print ' %s' % message
|
print ' %s' % message
|
||||||
count['warn'] += 1
|
count['warn'] += 1
|
||||||
|
|
||||||
def pwarn(message):
|
|
||||||
if options.pedantic:
|
|
||||||
warn(message)
|
|
||||||
|
|
||||||
# Parse command line...
|
# Parse command line...
|
||||||
parser = OptionParser(usage="Usage: %prog [options] [APPID [APPID ...]]")
|
parser = OptionParser(usage="Usage: %prog [options] [APPID [APPID ...]]")
|
||||||
parser.add_option("-v", "--verbose", action="store_true", default=False,
|
parser.add_option("-v", "--verbose", action="store_true", default=False,
|
||||||
help="Spew out even more information than normal")
|
help="Spew out even more information than normal")
|
||||||
parser.add_option("-q", "--quiet", action="store_true", default=False,
|
parser.add_option("-q", "--quiet", action="store_true", default=False,
|
||||||
help="Restrict output to warnings and errors")
|
help="Restrict output to warnings and errors")
|
||||||
parser.add_option("-p", "--pedantic", action="store_true", default=False,
|
|
||||||
help="Show pedantic warnings that might give false positives")
|
|
||||||
(options, args) = parser.parse_args()
|
(options, args) = parser.parse_args()
|
||||||
|
|
||||||
config = common.read_config(options)
|
config = common.read_config(options)
|
||||||
@ -138,7 +132,7 @@ def main():
|
|||||||
if not curbuild or int(build['vercode']) > int(curbuild['vercode']):
|
if not curbuild or int(build['vercode']) > int(curbuild['vercode']):
|
||||||
curbuild = build
|
curbuild = build
|
||||||
|
|
||||||
# Potentially incorrect UCM
|
# Incorrect UCM
|
||||||
if (curbuild and curbuild['commit']
|
if (curbuild and curbuild['commit']
|
||||||
and app['Update Check Mode'] == 'RepoManifest'
|
and app['Update Check Mode'] == 'RepoManifest'
|
||||||
and not curbuild['commit'].startswith('unknown')
|
and not curbuild['commit'].startswith('unknown')
|
||||||
@ -147,12 +141,6 @@ def main():
|
|||||||
warn("Last used commit '%s' looks like a tag, but Update Check Mode is '%s'" % (
|
warn("Last used commit '%s' looks like a tag, but Update Check Mode is '%s'" % (
|
||||||
curbuild['commit'], app['Update Check Mode']))
|
curbuild['commit'], app['Update Check Mode']))
|
||||||
|
|
||||||
# Dangerous auto updates
|
|
||||||
if curbuild and app['Auto Update Mode'] != 'None':
|
|
||||||
for flag in ['target', 'srclibs', 'scanignore']:
|
|
||||||
if curbuild[flag]:
|
|
||||||
pwarn("Auto Update Mode is enabled but '%s' is manually set at '%s'" % (flag, curbuild[flag]))
|
|
||||||
|
|
||||||
# Summary size limit
|
# Summary size limit
|
||||||
summ_chars = len(app['Summary'])
|
summ_chars = len(app['Summary'])
|
||||||
if summ_chars > config['char_limits']['Summary']:
|
if summ_chars > config['char_limits']['Summary']:
|
||||||
|
Loading…
Reference in New Issue
Block a user