easybuild-easyconfigs icon indicating copy to clipboard operation
easybuild-easyconfigs copied to clipboard

Remove whitespace at EOL from ECs

Open Flamefire opened this issue 5 years ago • 3 comments

First step of enabling flake8 for ECs.

Cmds run:

  • flake8 --filename '*.eb' --extend-ignore F821 -j4 --count to find and count issues excluding unknown variables
  • flake8 --filename '*.eb' --select W291 -j5 to find only EOL whitespace
  • autopep8 -i --select W291 --max-line-length 120 to fix only this issue, in particular: autopep8 -i --select W291 --max-line-length 120 $(flake8 --filename '*.eb' --select W291 -j5 --format='%(path)s' | sort -u)

Note that git diff develop --ignore-space-at-eol turns up empty except for the CI disabling

Flamefire avatar Aug 20 '20 14:08 Flamefire

Rebased, conflicts resolved and updated so flake8 --filename '*.eb' --select W291 -j5 turns up nothing.

Flamefire avatar Jul 26 '22 11:07 Flamefire

Rebased again, fixed newly introduced issues again.

Flamefire avatar Oct 18 '22 07:10 Flamefire

Rebased to include removing recently introduced whitespace and resolve conflicts

Flamefire avatar Oct 23 '23 08:10 Flamefire