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

Make usage of blank lines more consistent in edkrepo

Open kevinsun49 opened this issue 2 years ago • 2 comments

https://peps.python.org/pep-0008/#blank-lines

Ensure all code in edkrepo fits the following criteria:

  • Surround top-level function and class definitions with two blank lines.
  • Method definitions inside a class are surrounded by a single blank line.

In short, all classes should have 2 blank lines surrounding them, all global level functions should have 2 blank lines surrounding them, and all methods inside of a class should have 1 blank line surrounding them.

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