edk2-edkrepo icon indicating copy to clipboard operation
edk2-edkrepo copied to clipboard

Scan for line length violations in edkrepo

Open kevinsun49 opened this issue 2 years ago • 0 comments

https://peps.python.org/pep-0008/#maximum-line-length

There may be areas in the code where some lines are of a very long length (>120 characters). Propose shortening them by breaking into multiple lines via the following protocol:

  • Strong preference to use implicit line continuation inside () [] and {}
  • Expressions can be wrapped in ()
  • Otherwise, default to using a backslash character \ to indicate line break

Please submit one PR with one commit for each of the following categories:

  • edkrepo/commands/*
  • edkrepo/common/*
  • All of these together:
    • edkrepo/config/*
    • edkrepo/git_automation/*
    • edkrepo/command_completion_edkrepo.py
    • edkrepo/edkrepo_cli.py
    • edkrepo/edkrepo_entry_point.py
  • edkrepo_manifest_parser/edk_manifest.py

kevinsun49 avatar Oct 05 '22 18:10 kevinsun49