Update Slack channel for Daily PR's count Bot - #224
Conversation
🔍 Vulnerabilities of
|
| digest | sha256:fba1e17131803e93b99f50ffb8af489f31575b0bb1635bee6548ef6418917c19 |
| vulnerabilities | |
| platform | linux/amd64 |
| size | 291 MB |
| packages | 984 |
📦 Base Image node:23-alpine
| also known as |
|
| digest | sha256:b9d38d589853406ff0d4364f21969840c3e0397087643aef8eede40edbb6c7cd |
| vulnerabilities |
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description
| ||||||||||||
Description
|
|
WalkthroughUpdates the Slack channel value in .github/workflows/daily-pr-count.yml for the “Run PR counter script” step, changing the environment variable SLACK_CHANNEL from "#spdfn-website" to "#spdfn-website-updates". No other workflow logic, steps, or environment variables are modified. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR/Issue comments)Type Other keywords and placeholders
CodeRabbit Configuration File (
|
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (4)
.github/workflows/daily-pr-count.yml (4)
35-35: Verify Slack webhook/channel compatibility; prefer channel ID over name.
If the webhook is bound to a default channel, Slack will ignore an override, and the post may still go to the old channel. Also, many Slack APIs expect a channel ID (Cxxxxxxxx) instead of a name with '#'. Ensure the app is invited to #spdfn-website-updates and consider switching to the channel ID for robustness.Example diff (replace with the real ID):
- SLACK_CHANNEL: "#spdfn-website-updates" + SLACK_CHANNEL: "C0123456789"
5-6: Fix the UTC/CET comment (it’s inaccurate).
Cron '0 20 * * *' runs at 20:00 UTC, which is 21:00 CET (not 22:00/“10 PM CET”). Update the comment to avoid confusion, especially around DST.- # Run at 10 PM CET (8 PM UTC) + # Runs at 20:00 UTC (21:00 CET, 22:00 CEST)
19-24: Upgrade actions and Node to supported LTS.
Move to checkout/setup-node v4 and Node 20 LTS to stay on maintained versions.- uses: actions/checkout@v3 + uses: actions/checkout@v4 ... - uses: actions/setup-node@v3 + uses: actions/setup-node@v4 with: - node-version: 16 + node-version: 20
26-30: Invoke the Node script explicitly; avoid chmod + shebang dependency.
Calling node directly removes the need for executable bits and a shebang, reducing failure modes.- run: | - chmod +x ./scripts/merged-prs-last-24h.js - ./scripts/merged-prs-last-24h.js + run: | + node ./scripts/merged-prs-last-24h.js
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
💡 Knowledge Base configuration:
- MCP integration is disabled by default for public repositories
- Jira integration is disabled by default for public repositories
- Linear integration is disabled by default for public repositories
You can enable these sources in your CodeRabbit configuration.
📒 Files selected for processing (1)
.github/workflows/daily-pr-count.yml(1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: unit-tests
- GitHub Check: lint
- GitHub Check: e2e-tests
- GitHub Check: security-scan



No description provided.