add post_summary_to_slack workflow #81
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR adds a new GitHub Actions workflow for posting content summaries to Slack. The workflow provides a configurable pipeline that can scrape content from URLs, summarize it using different methods, and send notifications through various channels.
- Adds a manually triggered workflow with configurable scraper, summarizer, and notifier implementations
- Supports multiple content sources including web scraping and YouTube transcripts
- Integrates with Azure OpenAI and Slack through environment variables and secrets
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| inputs: | ||
| text: | ||
| type: string | ||
| default: "https://example.com,https://www.youtube.com/watch?v=xxx" |
There was a problem hiding this comment.
The default URLs include a placeholder YouTube URL with 'xxx' which could be confusing to users. Consider using a real example URL or a more descriptive placeholder like 'YOUTUBE_VIDEO_ID'.
| default: "https://example.com,https://www.youtube.com/watch?v=xxx" | |
| default: "https://example.com,https://www.youtube.com/watch?v=YOUTUBE_VIDEO_ID" |
Copilot uses AI. Check for mistakes.
| jobs: | ||
| post_summary_to_slack: | ||
| runs-on: "ubuntu-latest" | ||
| timeout-minutes: 5 |
There was a problem hiding this comment.
A 5-minute timeout may be insufficient for scraping multiple URLs and generating summaries, especially when using LLM-based summarization. Consider increasing the timeout to 10-15 minutes to prevent premature job cancellation.
| timeout-minutes: 5 | |
| timeout-minutes: 15 |
Copilot uses AI. Check for mistakes.
What this PR does / why we need it
Which issue(s) this PR fixes:
Fixes #80
Does this introduce a breaking change?
Pull Request Type
What kind of change does this Pull Request introduce?
How to Test
What to Check
Verify that the following are valid
Additional Information