Skip to content

Delete .github/workflows/fetch-x-feed.yml - #75

Merged
BreadMotion merged 1 commit into
mainfrom
BreadMotion-patch-5
Sep 16, 2026
Merged

BreadMotion merged 1 commit into
mainfrom
BreadMotion-patch-5

Conversation

@BreadMotion

Copy link
Copy Markdown
Owner

No description provided.

Signed-off-by: パン君 <yamazakimotionpg@gmail.com>
@BreadMotion BreadMotion self-assigned this Sep 16, 2026
Copilot AI lite review requested due to automatic review settings September 16, 2026 15:20
@BreadMotion
BreadMotion merged commit 1a68085 into main Sep 16, 2026
1 check passed

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟡 Changes recommended

Repository documentation currently references the workflow as “renamed to .disabled”, which becomes incorrect once the workflow file is deleted.

Get a fresh assessment by requesting another Copilot review.

Pull request overview

This PR removes the GitHub Actions workflow that periodically fetched an X (Twitter) media feed and committed WebSite/assets/x-feed.json updates back into the repository.

Changes:

  • Deletes .github/workflows/fetch-x-feed.yml, removing both the scheduled run and the manual workflow_dispatch entry point.
  • Eliminates the workflow’s dependency on the X_BEARER_TOKEN secret and associated commit automation.

Note (blocking for correctness of repo docs): Document/X_INTEGRATION.md still states that .github/workflows/fetch-x-feed.yml was “disabled by renaming to .disabled”, which is now inaccurate given the file is deleted.

File summaries
File Description
.github/workflows/fetch-x-feed.yml Removes the scheduled/manual workflow responsible for fetching X media and writing WebSite/assets/x-feed.json.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants