Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the news aggregator configuration and refreshes documentation/metadata for the AI-Pulse project. It adds an IoT feed category, refines the README’s branding and navigation, and introduces a simple .init marker file.
Changes:
- Add an
iotfeed category toFEED_CATEGORIESinsrc/aggregator.js. - Simplify and rework sections of
README.md(title, navigation table, footer wording, and links). - Add a new
.initfile containing a brief initialization note.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
src/aggregator.js |
Extends the feed configuration with an IoT category (currently syntactically misplaced outside the object literal). |
README.md |
Updates branding, navigation layout, and footer text (navigation table and footer line need markdown/wording fixes). |
.init |
Adds a one-line initialization marker (line content contains a duplicated word prefix). |
Comments suppressed due to low confidence (2)
src/aggregator.js:3
- Unused variable axios.
const axios = require('axios');
src/aggregator.js:4
- Unused variable Octokit.
const { Octokit } = require('@octokit/rest');
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1 @@ | |||
| .initInit commit before dev changes | |||
There was a problem hiding this comment.
The .init file content appears to contain a duplicated prefix (.initInit commit before dev changes), which reads like a typo rather than an intentional message. If this is meant to be a human-readable marker, consider simplifying it to something like Init commit before dev changes.
| .initInit commit before dev changes | |
| Init commit before dev changes |
| iot: [ | ||
| { name: 'IoT For All', url: 'https://www.iotforall.com/feed', tags: ['IoT', 'News'] }, | ||
| { name: 'IoT Business News', url: 'https://iotbusinessnews.com/feed/', tags: ['IoT', 'News'] }, | ||
| { name: 'IoT World Today', url: 'https://www.iotworldtoday.com/feed', tags: ['IoT', 'News'] }, | ||
| { name: 'Domotique News', url: 'https://www.domotiqueactualite.fr/feed/', tags: ['IoT', 'Domotique', 'News'] }, | ||
| { name: 'HomeTech', url: 'https://hometechmag.com/feed/', tags: ['IoT', 'Domotique', 'News'] } | ||
| ] |
There was a problem hiding this comment.
The new iot category is declared after the FEED_CATEGORIES object has already been closed and without a trailing comma after the previous property, which makes this file invalid JavaScript and prevents FEED_CATEGORIES.iot from ever being defined on the object. This iot block should be moved inside the FEED_CATEGORIES object literal and the preceding category entry should end with a comma so the object syntax remains valid.
|
|
||
| | 📚 [Repository](https://github.com/ThePhoenixAgency/AI-Pulse) | 👨💻 [Organization](https://github.com/ThePhoenixAgency) | 🔐 [Docs](./database/SUPABASE_MIGRATION.md) | | ||
| |:---:|:---:|:---:| | ||
| | [Repository](https://github.com/ThePhoenixAgency/AI-Pulse) | [Website](https://ThePhoenixAgency.github.io) ||:---:|:---:|:---:| |
There was a problem hiding this comment.
The navigation table markdown is malformed here: the header row and alignment row are combined on a single line and there is a mismatch between the number of header cells and data cells, so this is unlikely to render as an intended table. Please split the header and alignment into separate lines and ensure the same number of columns in both the header and data rows (including any docs/website column you want to keep).
| | [Repository](https://github.com/ThePhoenixAgency/AI-Pulse) | [Website](https://ThePhoenixAgency.github.io) ||:---:|:---:|:---:| | |
| | [Repository](https://github.com/ThePhoenixAgency/AI-Pulse) | [Website](https://ThePhoenixAgency.github.io) | Docs / Info | | |
| | :---: | :---: | :---: | |
| --- | ||
|
|
||
| <sub>*Powered by [AI-Pulse](https://github.com/ThePhoenixAgency/AI-Pulse) | 100% Free & Open Source | Built with ❤️ by ThePhoenixAgency*</sub> | ||
| <sub>*Powered by [AI-Pulse](https://github.com/ThePhoenixAgency/AI-Pulse) | 100% Free & Open Source | Built with by ThePhoenixAgency*</sub> |
There was a problem hiding this comment.
The phrase Built with by ThePhoenixAgency is missing a word, which reads awkwardly in the footer. Consider changing it to something like Built with love by ThePhoenixAgency to fix the grammar.
| <sub>*Powered by [AI-Pulse](https://github.com/ThePhoenixAgency/AI-Pulse) | 100% Free & Open Source | Built with by ThePhoenixAgency*</sub> | |
| <sub>*Powered by [AI-Pulse](https://github.com/ThePhoenixAgency/AI-Pulse) | 100% Free & Open Source | Built with love by ThePhoenixAgency*</sub> |
Initialize .init file with initial commit message
@EthanThePhoenix38
EthanThePhoenix38 authored last week
Fix formatting and update README.md content
@EthanThePhoenix38
EthanThePhoenix38 authored last week
Refactor README structure and update links
@EthanThePhoenix38
EthanThePhoenix38 authored last week
Add IoT news sources to aggregator
@EthanThePhoenix38
EthanThePhoenix38 authored last week