Skip to content

Commit e95e529

Browse files
committed
removing debug logs
1 parent 917f1d6 commit e95e529

2 files changed

Lines changed: 1 addition & 37 deletions

File tree

.github/workflows/verify-pr.yml

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -88,8 +88,6 @@ jobs:
8888
python-version: 3.12
8989
- run: npm install
9090
- name: Test changes in affected packages
91-
env:
92-
DEBUG_BULK_SUPPRESSIONS: ${{ matrix.os == 'windows-latest' && 'true' || 'false' }}
9391
run: |
9492
BASE_SHA=${{ github.event.pull_request.base.sha }}
9593
HEAD_SHA=${{ github.event.pull_request.head.sha }}

packages/code-analyzer-core/src/suppressions/bulk-suppression-processor.ts

Lines changed: 1 addition & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -135,49 +135,22 @@ function doesFileMatchConfigPath(
135135
configPath: string,
136136
workspaceRoot: string
137137
): boolean {
138-
const debugEnabled = process.env.DEBUG_BULK_SUPPRESSIONS === 'true';
139-
140-
if (debugEnabled) {
141-
console.log('[DEBUG] doesFileMatchConfigPath called:');
142-
console.log(' violationFile:', violationFile);
143-
console.log(' configPath:', configPath);
144-
console.log(' workspaceRoot:', workspaceRoot);
145-
console.log(' path.isAbsolute(configPath):', path.isAbsolute(configPath));
146-
}
147-
148138
// Config paths can be relative (recommended, like .gitignore) or absolute
149139
// If relative, resolve against workspace root
150140
// If absolute, use as-is
151141
const absoluteConfigPath = path.resolve(workspaceRoot, configPath);
152142

153-
if (debugEnabled) {
154-
console.log(' After path.resolve:', absoluteConfigPath);
155-
}
156-
157143
// Normalize paths for consistent comparison
158144
const normalizedViolationFile = path.normalize(violationFile);
159145
const normalizedConfigPath = path.normalize(absoluteConfigPath);
160146

161-
if (debugEnabled) {
162-
console.log(' After normalize - violation:', normalizedViolationFile);
163-
console.log(' After normalize - config:', normalizedConfigPath);
164-
}
165-
166147
// Normalize to POSIX separators for cross-platform comparison
167148
// Convert all backslashes to forward slashes for consistent comparison
168149
const comparisonViolationFile = normalizedViolationFile.replace(/\\/g, '/');
169150
const comparisonConfigPath = normalizedConfigPath.replace(/\\/g, '/');
170151

171-
if (debugEnabled) {
172-
console.log(' After POSIX normalize - violation:', comparisonViolationFile);
173-
console.log(' After POSIX normalize - config:', comparisonConfigPath);
174-
}
175-
176152
// Check if it's an exact file match
177153
if (comparisonViolationFile === comparisonConfigPath) {
178-
if (debugEnabled) {
179-
console.log(' Result: EXACT MATCH');
180-
}
181154
return true;
182155
}
183156

@@ -188,14 +161,7 @@ function doesFileMatchConfigPath(
188161
? comparisonConfigPath
189162
: comparisonConfigPath + '/';
190163

191-
const result = comparisonViolationFile.startsWith(configPathWithSep);
192-
193-
if (debugEnabled) {
194-
console.log(' Folder match check - configPathWithSep:', configPathWithSep);
195-
console.log(' Result:', result ? 'FOLDER MATCH' : 'NO MATCH');
196-
}
197-
198-
return result;
164+
return comparisonViolationFile.startsWith(configPathWithSep);
199165
}
200166

201167
/**

0 commit comments

Comments
 (0)