Skip to content

Commit

Permalink
fix: fix the pi-check command with lint flag
Browse files Browse the repository at this point in the history
  • Loading branch information
salman2013 committed Dec 2, 2024
1 parent 96d1235 commit 8c1f5b9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
2 changes: 0 additions & 2 deletions .annotation_safe_list.yml
Original file line number Diff line number Diff line change
Expand Up @@ -142,8 +142,6 @@ workflow.AssessmentWorkflowStep:
# Via edx-celeryutils
celery_utils.ChordData:
".. no_pii:": "No PII"
celery_utils.FailedTask:
".. no_pii:": "No PII"

# Via completion XBlock
completion.BlockCompletion:
Expand Down
6 changes: 4 additions & 2 deletions Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -223,13 +223,15 @@ pii_check: ## check django models for pii annotations
code_annotations django_find_annotations \
--config_file .pii_annotations.yml \
--app_name cms \
--coverage
--coverage \
--lint

DJANGO_SETTINGS_MODULE=lms.envs.test \
code_annotations django_find_annotations \
--config_file .pii_annotations.yml \
--app_name lms \
--coverage
--coverage \
--lint

check_keywords: ## check django models for reserve keywords
DJANGO_SETTINGS_MODULE=cms.envs.test \
Expand Down

0 comments on commit 8c1f5b9

Please sign in to comment.