openedx / openedx/openedx-platform
pii_check may not really be checking
Open
@kdmccormick is already working on this.
Since May 14, 2026.
- Dominant language
- Python
- Stars
- 8.2k
- Forks
- 4.4k
- Avg merge
- 6d 18h
- Merged PRs (30d)
- 42
Description
Here is the code for pii_check:
- https://github.com/openedx/openedx-platform/blob/7c68f1d255b3d56c99e6ea9e73f484e5aae5957e/Makefile#L177-L191
- This uses
--app_name cms, which doesn't seem to be matching anything. - Also, line 177 has comment about "Re-enable --lint flag", which seems to already be done?
- This uses
Possibly the target might be missed, given we don't seem to really have been checking?
Relevant recent issues and docs:
- https://github.com/openedx/openedx-platform/issues/35775
- https://docs.openedx.org/projects/edx-platform/en/latest/decisions/0021-fixing-quality-and-js-checks.html#pii-check
In addition to fixing, maybe we could implement or ticket ensuring this doesn't regress again? It seems this isn't the first time this broke and no one noticed.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Assessment
This issue has not been assessed yet.