Skip to content

Commit 35126c0

Browse files
dreamiurgclaude
andauthored
chore: remove ineffective lgtm suppression comments (#58)
Remove lgtm[...] syntax from inline comments as it doesn't work with GitHub CodeQL. The old LGTM.com suppression syntax is not supported by GitHub's Code Scanning. The explanatory comments are retained as documentation for why the code is structured this way, even though CodeQL alerts must be dismissed via the GitHub UI rather than inline comments. All related CodeQL false positive alerts (31-35) have been dismissed via the GitHub API with appropriate explanations. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: Claude <noreply@anthropic.com>
1 parent 6f5d958 commit 35126c0

File tree

1 file changed

+4
-6
lines changed

1 file changed

+4
-6
lines changed

peakbagger/formatters.py

Lines changed: 4 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,6 @@ def _print_json(self, data: dict[str, Any] | list[dict[str, Any]]) -> None:
9494
Uses plain print() instead of console.print() to avoid Rich processing
9595
escape sequences in the JSON, which would make it invalid.
9696
"""
97-
# lgtm[py/clear-text-logging-sensitive-data]
9897
# This is intentional CLI output, not logging. The tool only handles public
9998
# peak data from PeakBagger.com - no credentials or sensitive information.
10099
print(json.dumps(data, indent=2, ensure_ascii=False))
@@ -348,11 +347,10 @@ def strip_emojis(text: str) -> str:
348347
"""Remove emoji characters from text."""
349348
import re
350349

351-
# Pattern matches emoji ranges
352-
# lgtm[py/overly-large-range]
353-
# This regex is for display formatting only (removing emojis from scraped
354-
# text). Overlapping ranges are intentional for comprehensive emoji coverage.
355-
# Not used for security validation or input sanitization.
350+
# Pattern matches emoji ranges. This regex is for display formatting only
351+
# (removing emojis from scraped text). Overlapping ranges are intentional
352+
# for comprehensive emoji coverage. Not used for security validation or
353+
# input sanitization.
356354
emoji_pattern = re.compile(
357355
"["
358356
"\U0001f600-\U0001f64f" # emoticons

0 commit comments

Comments
 (0)