fix(lint): convert custom noqa directives to regular comments
Ruff only accepts standard rule codes (e.g., E712, F401) in noqa directives. Custom architecture validator codes (SEC-034, SVC-006, MOD-004, API-007) are now regular comments instead. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
@@ -1336,7 +1336,7 @@ class EmailService:
|
||||
related_id=related_id,
|
||||
)
|
||||
self.db.add(log)
|
||||
self.db.commit() # noqa: SVC-006 - Email logs are side effects, commit immediately
|
||||
self.db.commit() # SVC-006 - Email logs are side effects, commit immediately
|
||||
return log
|
||||
|
||||
# Inject branding variables if requested
|
||||
@@ -1471,7 +1471,7 @@ class EmailService:
|
||||
if not email_enabled:
|
||||
log.status = EmailStatus.FAILED.value
|
||||
log.error_message = "Email sending is disabled"
|
||||
self.db.commit() # noqa: SVC-006 - Email logs are side effects, commit immediately
|
||||
self.db.commit() # SVC-006 - Email logs are side effects, commit immediately
|
||||
logger.info(f"Email sending disabled, skipping: {to_email}")
|
||||
return log
|
||||
|
||||
@@ -1497,7 +1497,7 @@ class EmailService:
|
||||
log.mark_failed(error or "Unknown error")
|
||||
logger.error(f"Email failed to {to_email}: {error}")
|
||||
|
||||
self.db.commit() # noqa: SVC-006 - Email logs are side effects, commit immediately
|
||||
self.db.commit() # SVC-006 - Email logs are side effects, commit immediately
|
||||
return log
|
||||
|
||||
|
||||
|
||||
Reference in New Issue
Block a user