The pii_check in edx-platform not running for a long time. We want to re-enable it as part of:
However, running code_annotations with the --lint flag currently raises errors due to some malformed PII annotations. In particular, these annotations from edx-proctoring need to be fixed:
So, in the PR linked above, we have removed the --lint flag from make pii_check. This issue exists as a reminder for us to put the --lint flag back once the edx-proctoring issue, and any other pii linting errors, are resolved.
Please note that this is separate from @bmtcril 's effort to raise the PII annotation threshold, which is checked by nature of us passing the flag --coverage to code_annotations:
The pii_check in edx-platform not running for a long time. We want to re-enable it as part of:
However, running
code_annotationswith the--lintflag currently raises errors due to some malformed PII annotations. In particular, these annotations from edx-proctoring need to be fixed:pii_retirement: to_be_implementededx-proctoring#1241So, in the PR linked above, we have removed the
--lintflag frommake pii_check. This issue exists as a reminder for us to put the--lintflag back once the edx-proctoring issue, and any other pii linting errors, are resolved.Please note that this is separate from @bmtcril 's effort to raise the PII annotation threshold, which is checked by nature of us passing the flag
--coveragetocode_annotations: