maxtext icon indicating copy to clipboard operation
maxtext copied to clipboard

Prepare for methodological pylint resolutions

Open SamuelMarks opened this issue 7 months ago • 0 comments

Description

This PR is a branch from which my next PRs are based upon.

Currently this line:

pylint --disable C0301,C3001,C0114,C0115,C0116,C0200,C0121,C0201,C0206,C0209,C0412,C0415,C2801,E0102,E0606,E0611,E1102,E1111,E1120,E1121,E1123,E1135,E1136,R0401,R1701,R1703,R1710,R1711,R1735,R0917,R1714,R1716,R1719,R1721,R1728,R1728,W0102,W0107,W0201,W0212,W0221,W0223,W0237,W0404,W0611,W0612,W0613,W0621,W0622,W0631,W0707,W0718,W1201,W1203,W1309,W1514,W4901 MaxText/

Is being methodologically addressed in branches on my fork, explicitly:

  • lint-c for C0301,C3001,C0114,C0115,C0116,C0200,C0121,C0201,C0206,C0209,C0412,C0415,C2801 @ #1668
  • lint-e for E0102,E0606,E0611,E1102,E1111,E1120,E1121,E1123,E1135,E1136 @ #1700
  • lint-r for R0401,R1701,R1703,R1710,R1711,R1735,R0917,R1714,R1716,R1719,R1721,R1728,R1728 @ #1740
  • lint-w for W0102,W0107,W0201,W0212,W0221,W0223,W0237,W0404,W0611,W0612,W0613,W0621,W0622,W0631,W0707,W0718,W1201,W1203,W1309,W1514,W4901 @ #1749

So feel free to merge this branch at any time and the other PRs ASAP

Tests

N/A

Checklist

Before submitting this PR, please make sure (put X in square brackets):

  • [x] I have performed a self-review of my code.
  • [x] I have necessary comments in my code, particularly in hard-to-understand areas.
  • [x] I have run end-to-end tests tests and provided workload links above if applicable.
  • [x] I have made or will make corresponding changes to the doc if needed.

SamuelMarks avatar May 01 '25 04:05 SamuelMarks