Skip to content

update deps - #431

Merged
coryrylan merged 4 commits into
mainfrom
topic-update-deps
Oct 21, 2026
Merged

coryrylan merged 4 commits into
mainfrom
topic-update-deps

Conversation

@coryrylan

@coryrylan coryrylan commented Oct 20, 2026 •

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • Chores
    • Updated bundled software and license attributions to reflect newer ESLint, JSON linting, and HTML sanitization releases.
    • Updated HTML sanitization support used by internal tooling and related applications.
    • Updated Angular runtime and compiler packages to version 22.2.1 in the Angular starter project.
    • Automatic approval of edit and write permission requests is no longer enabled.
    • No public APIs or exported entities were changed.

@coderabbitai

coderabbitai Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/elements/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: 3db70d3c-6d57-47dd-805e-810811e30c5d
📥 Commits

Reviewing files that changed from the base of the PR and between 5827b28 and 8a34474.

📒 Files selected for processing: 1
  • .claude/settings.json
+------------------------------------------------------------------------+
| I'm CodeRabbit. I review the code you never knew you needed reviewing. |
+------------------------------------------------------------------------+
  \
   \   (\__/)
       (•ㅅ•)
       /   づ

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration
  • Configuration used: Repository: NVIDIA/elements/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Enterprise
  • Run ID: 2c14c244-1496-46ee-be4a-b988da9ba24e
📥 Commits

Reviewing files that changed from the base of the PR and between e38bd8a and 5827b28.

⛔ Files ignored due to path filters: 1
  • pnpm-lock.yaml is excluded by !**/pnpm-lock.yaml
📒 Files selected for processing: 7
  • NOTICE.md
  • projects/cli/NOTICE.md
  • projects/internals/tools/package.json
  • projects/lint/NOTICE.md
  • projects/pi/NOTICE.md
  • projects/pi/package.json
  • projects/starters/angular/package.json

Included review availability: This review used your included allowance. Your plan provides up to 12 included reviews per hour; 10 remain after this review.

📝 Walkthrough

The pull request updates sanitize-html and Angular dependency versions in package manifests. It also updates bundled software and license notices for @eslint/json, eslint, and sanitize-html.

Changes: Dependency updates

Layer / File(s) Summary
Update dependency versions
projects/internals/tools/package.json, projects/pi/package.json, projects/starters/angular/package.json
Updates sanitize-html from 2.17.7 to 2.18.0 in two manifests. Updates Angular runtime dependencies and @angular/compiler-cli from 22.2.0 to 22.2.1.
Align bundled software notices
NOTICE.md, projects/cli/NOTICE.md, projects/lint/NOTICE.md, projects/pi/NOTICE.md
Updates the listed versions and license attributions for @eslint/json, eslint, and sanitize-html.

Priority: ⬇️ Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: cursoragent

Merge Risk: ⚪ Minimal · up to 5827b

This is a routine dependency refresh with matching license-notice updates. The Angular starter keeps a supported toolchain combination, and no concrete merge-blocking risk remains.

Pre-merge checks: Passed 3 · Inconclusive 1

Failed Failed checks: 1 inconclusive

Check name Status Explanation Resolution
Title check Inconclusive The title is related to the dependency updates, but "update deps" is too vague to identify the affected packages or primary change. Use a specific title such as "Update ESLint, sanitize-html, and Angular dependencies".
Passed Passed checks: 3 passed
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

Signed-off-by: Cory Rylan <[email protected]>
Signed-off-by: Cory Rylan <[email protected]>
Signed-off-by: Cory Rylan <[email protected]>
@coryrylan coryrylan changed the title Topic update deps update deps Oct 20, 2026
@github-code-quality

github-code-quality Bot commented Oct 20, 2026 •

Copy link
Copy Markdown

Code coverage overview

Languages: TypeScript

TypeScript / code-coverage/vitest

main 36caeaf topic-update-deps 8a34474 +/-
99% 99% 0%

ℹ️ Calculations represent line coverage. Results may change as calculations are updated.


Updated October 21, 2026 15:56 UTC

@johnyanarella
johnyanarella self-requested a review October 21, 2026 15:47
@coryrylan
coryrylan enabled auto-merge (rebase) October 21, 2026 15:49
@coryrylan
coryrylan merged commit b4df8f3 into main Oct 21, 2026
15 checks passed
@coryrylan
coryrylan deleted the topic-update-deps branch October 21, 2026 16:02
@coryrylan

Copy link
Copy Markdown
Collaborator Author

🎉 This issue has been resolved in version 2.2.11 🎉

Changelog

@coryrylan

Copy link
Copy Markdown
Collaborator Author

🎉 This issue has been resolved in version 1.0.2 🎉

Changelog

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants