Skip to content

docs: removals from books and underlying tech to external lists#2152

Merged
jdegand merged 2 commits into
PatrickJS:gh-pagesfrom
jdegand:various-50
Apr 28, 2026
Merged

docs: removals from books and underlying tech to external lists#2152
jdegand merged 2 commits into
PatrickJS:gh-pagesfrom
jdegand:various-50

Conversation

@jdegand
Copy link
Copy Markdown
Collaborator

@jdegand jdegand commented Apr 28, 2026

Summary by CodeRabbit

  • Documentation
    • Updated curated resource lists with new book recommendations and clearer free resource labeling.
    • Refreshed cheat sheets section to reference the latest official version.
    • Improved library references across multiple categories with updated links and removed outdated entries.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Apr 28, 2026

📝 Walkthrough

Walkthrough

This PR updates the README.md documentation by adding and removing book entries, updating the cheat sheets section reference, pruning library entries across multiple sections (Visual Effects, RxJS, TypeScript, and others), and changing the Partytown URL to a new domain.

Changes

Cohort / File(s) Summary
Documentation Updates
README.md
Added "Code with Ahsan" and "Angular Enterprise Architecture" books; updated "Testing Angular" to explicitly mark as FREE; removed several book entries; updated cheat sheets reference to "Official Angular Version 17 Cheatsheet"; removed multiple library entries across Visual Effects, RxJS, TypeScript, cross-framework integration, external integration, Angular wrappers, and Angular-inspired solutions sections; updated Partytown URL to new domain.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

Poem

🐰 Hop through the lists, old entries we clear,
New books and cheatsheets make knowledge so dear,
URLs refreshed with a hop and a bound,
The finest resources are all that we've found!

🚥 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 clearly describes the main change: removing items from books and underlying tech sections to move them to external lists.
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

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.

❤️ Share

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

Copy link
Copy Markdown
Contributor

@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.

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
README.md (1)

397-406: ⚠️ Potential issue | 🟡 Minor

Make the “Books” entries consistent (avoid link-only items + consider “FREE” staleness).

Line 397 (“Code with Ahsan”) is a link-only bullet, while most surrounding entries use - descriptions and/or author attribution. Also, several entries use a hard-coded **FREE** label; if pricing changes later, the README will be misleading.

♻️ Proposed fixes
-* [Code with Ahsan](https://www.codewithahsan.dev/books)
+* [Code with Ahsan](https://www.codewithahsan.dev/books) - Angular books and ebooks.

(Optionally: for the **FREE** items, consider wording like “Free ebook (verify pricing)” or adding an “as of YYYY-MM-DD” note to reduce staleness risk.)

🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed.

In `@README.md` around lines 397 - 406, Make the "Books" list consistent by
converting the link-only bullet for "Code with Ahsan" into the same format used
by other entries (include a short description and author attribution where
available, e.g., "Code with Ahsan - Ahsan [short description]"), and replace
hard-coded "**FREE**" labels with a non-stale phrasing such as "Free ebook
(verify pricing)" or "Free as of YYYY-MM-DD" for each item that currently shows
**FREE** so the README remains accurate over time.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.

Outside diff comments:
In `@README.md`:
- Around line 397-406: Make the "Books" list consistent by converting the
link-only bullet for "Code with Ahsan" into the same format used by other
entries (include a short description and author attribution where available,
e.g., "Code with Ahsan - Ahsan [short description]"), and replace hard-coded
"**FREE**" labels with a non-stale phrasing such as "Free ebook (verify
pricing)" or "Free as of YYYY-MM-DD" for each item that currently shows **FREE**
so the README remains accurate over time.

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: b5c41775-6376-4352-8804-609ce5f11c03

📥 Commits

Reviewing files that changed from the base of the PR and between 476d854 and 41642d4.

📒 Files selected for processing (1)
  • README.md

@jdegand jdegand merged commit 2e6fa6e into PatrickJS:gh-pages Apr 28, 2026
1 check passed
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