github/kubernetes
1
0
Fork 0
mirror of https://github.com/k3s-io/kubernetes.git synced 2025-07-26 21:17:23 +00:00
Code Issues Projects Releases Wiki Activity

Merge pull request #82240 from keerthigd/fmt-pythonfile

Browse Source 
hack/boilerplate/bolierplate.py: format python file according to PEP8…
This commit is contained in:
Kubernetes Prow Robot 2019-09-11 10:41:33 -07:00 committed by GitHub
commit 54cae98f5f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 24 additions and 8 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
hack/boilerplate/boilerplate.py
View File

@ -48,6 +48,7 @@ args = parser.parse_args()
verbose_out = sys.stderr if args.verbose else open("/dev/null", "w") verbose_out = sys.stderr if args.verbose else open("/dev/null", "w")
def get_refs(): def get_refs():
refs = {} refs = {}
@ -61,6 +62,7 @@ def get_refs():
return refs return refs
def is_generated_file(filename, data, regexs): def is_generated_file(filename, data, regexs):
for d in skipped_ungenerated_files: for d in skipped_ungenerated_files:
if d in filename: if d in filename:
@ -69,6 +71,7 @@ def is_generated_file(filename, data, regexs):
p = regexs["generated"] p = regexs["generated"]
return p.search(data) return p.search(data)
def file_passes(filename, refs, regexs): def file_passes(filename, refs, regexs):
try: try:
f = open(filename, 'r') f = open(filename, 'r')
@ -119,9 +122,11 @@ def file_passes(filename, refs, regexs):
for d in data: for d in data:
if p.search(d): if p.search(d):
if generated: if generated:
print('File %s has the YEAR field, but it should not be in generated file' % filename, file=verbose_out) print('File %s has the YEAR field, but it should not be in generated file' %
filename, file=verbose_out)
else: else:
print('File %s has the YEAR field, but missing the year of date' % filename, file=verbose_out) print('File %s has the YEAR field, but missing the year of date' %
filename, file=verbose_out)
return False return False
if not generated: if not generated:
@ -134,7 +139,8 @@ def file_passes(filename, refs, regexs):
# if we don't match the reference at this point, fail # if we don't match the reference at this point, fail
if ref != data: if ref != data:
print("Header in %s does not match reference, diff:" % filename, file=verbose_out) print("Header in %s does not match reference, diff:" %
filename, file=verbose_out)
if args.verbose: if args.verbose:
print(file=verbose_out) print(file=verbose_out)
for line in difflib.unified_diff(ref, data, 'reference', filename, lineterm=''): for line in difflib.unified_diff(ref, data, 'reference', filename, lineterm=''):
@ -144,15 +150,19 @@ def file_passes(filename, refs, regexs):
return True return True
def file_extension(filename): def file_extension(filename):
return os.path.splitext(filename)[1].split(".")[-1].lower() return os.path.splitext(filename)[1].split(".")[-1].lower()
skipped_dirs = ['Godeps', 'third_party', '_gopath', '_output', '.git', 'cluster/env.sh', skipped_dirs = ['Godeps', 'third_party', '_gopath', '_output', '.git', 'cluster/env.sh',
"vendor", "test/e2e/generated/bindata.go", "hack/boilerplate/test", "vendor", "test/e2e/generated/bindata.go", "hack/boilerplate/test",
"staging/src/k8s.io/kubectl/pkg/generated/bindata.go"] "staging/src/k8s.io/kubectl/pkg/generated/bindata.go"]
# list all the files contain 'DO NOT EDIT', but are not generated # list all the files contain 'DO NOT EDIT', but are not generated
skipped_ungenerated_files = ['hack/lib/swagger.sh', 'hack/boilerplate/boilerplate.py'] skipped_ungenerated_files = [
'hack/lib/swagger.sh', 'hack/boilerplate/boilerplate.py']
def normalize_files(files): def normalize_files(files):
newfiles = [] newfiles = []
@ -165,6 +175,7 @@ def normalize_files(files):
newfiles[i] = os.path.join(args.rootdir, pathname) newfiles[i] = os.path.join(args.rootdir, pathname)
return newfiles return newfiles
def get_files(extensions): def get_files(extensions):
files = [] files = []
if len(args.filenames) > 0: if len(args.filenames) > 0:
@ -192,25 +203,29 @@ def get_files(extensions):
outfiles.append(pathname) outfiles.append(pathname)
return outfiles return outfiles
def get_dates(): def get_dates():
years = datetime.datetime.now().year years = datetime.datetime.now().year
return '(%s)' % '|'.join((str(year) for year in range(2014, years+1))) return '(%s)' % '|'.join((str(year) for year in range(2014, years+1)))
def get_regexs(): def get_regexs():
regexs = {} regexs = {}
# Search for "YEAR" which exists in the boilerplate, but shouldn't in the real thing # Search for "YEAR" which exists in the boilerplate, but shouldn't in the real thing
regexs["year"] = re.compile( 'YEAR' ) regexs["year"] = re.compile('YEAR')
# get_dates return 2014, 2015, 2016, 2017, or 2018 until the current year as a regex like: "(2014|2015|2016|2017|2018)"; # get_dates return 2014, 2015, 2016, 2017, or 2018 until the current year as a regex like: "(2014|2015|2016|2017|2018)";
# company holder names can be anything # company holder names can be anything
regexs["date"] = re.compile(get_dates()) regexs["date"] = re.compile(get_dates())
# strip // +build \n\n build constraints # strip // +build \n\n build constraints
regexs["go_build_constraints"] = re.compile(r"^(// \+build.*\n)+\n", re.MULTILINE) regexs["go_build_constraints"] = re.compile(
r"^(// \+build.*\n)+\n", re.MULTILINE)
# strip #!.* from scripts # strip #!.* from scripts
regexs["shebang"] = re.compile(r"^(#!.*\n)\n*", re.MULTILINE) regexs["shebang"] = re.compile(r"^(#!.*\n)\n*", re.MULTILINE)
# Search for generated files # Search for generated files
regexs["generated"] = re.compile( 'DO NOT EDIT' ) regexs["generated"] = re.compile('DO NOT EDIT')
return regexs return regexs
def main(): def main():
regexs = get_regexs() regexs = get_regexs()
refs = get_refs() refs = get_refs()
@ -222,5 +237,6 @@ def main():
return 0 return 0
if __name__ == "__main__": if __name__ == "__main__":
sys.exit(main()) sys.exit(main())