Skip to content

Commit 9c8e587

Browse files
slack-notify-pr: add support for defining exact slackStream
1 parent 611903f commit 9c8e587

File tree

1 file changed

+3
-20
lines changed

1 file changed

+3
-20
lines changed

slack-notify-pr/action.yml

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -52,51 +52,34 @@ runs:
5252
let streamPathMap = new Map();
5353
5454
const slackStream = process.env.SLACK_STREAM;
55-
console.log(`DEBUG: slackStream input value: "${slackStream}"`);
56-
console.log(`DEBUG: slackStream input length: ${slackStream ? slackStream.length : 0}`);
57-
console.log(`DEBUG: slackStream input type: ${typeof slackStream}`);
58-
console.log(`DEBUG: slackStream input truthy: ${!!slackStream}`);
5955
if (slackStream) {
6056
slackStreams.add(slackStream);
61-
console.log(`DEBUG: Added direct slackStream: ${slackStream}`);
62-
} else {
63-
console.log(`DEBUG: slackStream is falsy, not adding to streams`);
6457
}
6558
6659
const pathToSlackStreamMapping = process.env.PATH_TO_SLACK_STREAM_MAPPING;
67-
console.log(`DEBUG: pathToSlackStreamMapping input value: "${pathToSlackStreamMapping}"`);
6860
if (pathToSlackStreamMapping) {
6961
try {
7062
const parsedMapping = JSON.parse(pathToSlackStreamMapping);
71-
console.log(`DEBUG: Parsed mapping:`, JSON.stringify(parsedMapping, null, 2));
72-
7363
const files = await github.paginate(github.rest.pulls.listFiles, {
7464
owner: context.repo.owner,
7565
repo: context.repo.repo,
7666
pull_number: pr.number,
7767
});
78-
console.log(`DEBUG: Found ${files.length} changed files.`);
79-
console.log(`DEBUG: Changed files:`, files.map(f => f.filename));
80-
68+
console.log(`Found ${files.length} changed files.`);
8169
for (const file of files) {
8270
for (const [path, slackStream] of Object.entries(parsedMapping)) {
83-
console.log(`DEBUG: Checking if file "${file.filename}" starts with path "${path}"`);
8471
if (file.filename.startsWith(path)) {
85-
console.log(`DEBUG: Match found! Adding stream for: ${file.filename} -> ${slackStream}`);
72+
console.log(`Add stream for: ${file.filename}`)
8673
slackStreams.add(slackStream);
8774
streamPathMap.set(slackStream, path);
8875
}
8976
}
9077
}
9178
} catch (error) {
92-
console.error('DEBUG: Failed to parse pathToSlackStreamMapping JSON:', error);
93-
console.log('DEBUG: Skipping path-based stream mapping due to invalid JSON');
79+
console.error('Failed to parse pathToSlackStreamMapping JSON:', error);
9480
}
9581
}
9682
97-
console.log(`DEBUG: Final slackStreams size: ${slackStreams.size}`);
98-
console.log(`DEBUG: Final slackStreams:`, Array.from(slackStreams));
99-
10083
if (slackStreams.size === 0) {
10184
console.log("No Slack streams configured for notification.");
10285
return;

0 commit comments

Comments
 (0)