Skip to content

Auto PR: Scheduled addition of new sources#22

Merged
semmet95 merged 1 commit into
mainfrom
src-auto-add-25617971999
May 10, 2026
Merged

Auto PR: Scheduled addition of new sources#22
semmet95 merged 1 commit into
mainfrom
src-auto-add-25617971999

Conversation

@semmet95
Copy link
Copy Markdown
Contributor

@semmet95 semmet95 commented May 10, 2026

Adds most popular news source in the last week for ingestion

Summary by CodeRabbit

Release Notes

  • New Features
    • Added 8 new news sources: CNN, Fox News, Google News, India Times, MSN, NDTV, The Guardian, and Yahoo Finance, expanding global news coverage across multiple regions and categories.

Review Change Stack

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented May 10, 2026

📝 Walkthrough

Walkthrough

This PR adds eight new news source definitions to the sources/ directory. Each file follows a consistent YAML structure defining a source with name, summary, regional or global tags, and URI—covering American (CNN, Fox News, Google News, MSN, Yahoo Finance), Indian (India Times, NDTV), and British (The Guardian) outlets.

Changes

News Source Definitions

Layer / File(s) Summary
Source Definitions
sources/CNN.yaml, sources/Fox-News.yaml, sources/Google-News.yaml, sources/India-Times.yaml, sources/MSN.yaml, sources/NDTV.yaml, sources/The-Guardian.yaml, sources/Yahoo-Finance.yaml
Eight new YAML source definitions added with identical structure: name, descriptive summary, regional/global tags (american, british, indian, or global), and uri pointing to each source's homepage.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

  • SatyaLens/sources#20: Introduces the workflow and automation that generates and opens PRs to add source YAML definitions like those in this PR.

Poem

🐰 Eight sources leap into view,
From CNN to Guardian true,
Tags and URIs neat and tidy,
Global news from every side-y!
YAML files, a hoppy delight. ✨

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'Auto PR: Scheduled addition of new sources' accurately describes the main change—adding 8 new news source definitions to the sources directory.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch src-auto-add-25617971999

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@sources/India-Times.yaml`:
- Around line 1-4: Update the source identity fields so they consistently refer
to the same publication: change the name, summary and uri to all reference "The
Times of India" (or alternatively change summary/uri to match "India Times");
specifically update the `name` field and, if keeping The Times of India, set
`name: "The Times of India"`, ensure `summary` mentions The Times of India, and
keep `uri: "https://timesofindia.indiatimes.com"` so `name`, `summary`, and
`uri` are aligned for the source definition.

In `@sources/NDTV.yaml`:
- Around line 1-4: The Source document is missing the required properties
"score" and "uriDigest"; update the NDTV YAML to include a numeric score (e.g.,
a float between 0.0 and 1.0 in the "score" field) and a "uriDigest" field
containing a stable digest of the "uri" (e.g., the hex SHA-256 of
"https://www.ndtv.com"); add these two fields alongside the existing
name/summary/tags/uri so the file conforms to the Source schema.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: afa260c7-06e1-45e7-95b9-dd484fc92ae2

📥 Commits

Reviewing files that changed from the base of the PR and between 0bc8524 and 96d0fa8.

📒 Files selected for processing (8)
  • sources/CNN.yaml
  • sources/Fox-News.yaml
  • sources/Google-News.yaml
  • sources/India-Times.yaml
  • sources/MSN.yaml
  • sources/NDTV.yaml
  • sources/The-Guardian.yaml
  • sources/Yahoo-Finance.yaml

Comment thread sources/India-Times.yaml
Comment thread sources/NDTV.yaml
Copy link
Copy Markdown

@cubic-dev-ai cubic-dev-ai Bot left a comment

Choose a reason for hiding this comment

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

No issues found across 8 files

@semmet95 semmet95 merged commit 0b79e2b into main May 10, 2026
3 checks passed
@semmet95 semmet95 deleted the src-auto-add-25617971999 branch May 10, 2026 12:21
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.

1 participant