Skip to content

#85 - update to angular 22#86

Open
mumenthalers wants to merge 3 commits into
mainfrom
#85-update-to-angular-22
Open

#85 - update to angular 22#86
mumenthalers wants to merge 3 commits into
mainfrom
#85-update-to-angular-22

Conversation

@mumenthalers

Copy link
Copy Markdown
Contributor

closes #85

Copilot AI review requested due to automatic review settings July 3, 2026 12:53
@mumenthalers

Copy link
Copy Markdown
Contributor Author

@copilot implement issue #85

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.

Pull request overview

Adds documentation to define Angular version support strategy and standardize terminology across the monorepo, in the context of the planned Angular 22 upgrade.

Changes:

  • Add an ADR documenting the “one Angular major per library major” policy and rationale.
  • Add a CONTEXT.md glossary defining preferred terms (Library, Consumer, Styleguide, Angular Version Support).

Reviewed changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated 2 comments.

File Description
docs/adr/0001-one-angular-major-per-library-major.md New ADR documenting Angular major ↔ library major alignment and why dual peer ranges are avoided.
CONTEXT.md New project context/glossary to standardize terminology in docs and PR discussions.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread docs/adr/0001-one-angular-major-per-library-major.md Outdated
Comment thread docs/adr/0001-one-angular-major-per-library-major.md Outdated

Copilot AI commented Jul 3, 2026

Copy link
Copy Markdown

@copilot implement issue #85

Implemented in commit 5693786: upgraded the monorepo to Angular 22 (root Angular deps/tooling + library peerDependencies), regenerated the lockfile, and aligned the ADR terminology.

@mumenthalers mumenthalers left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

revert .npmrc change

Comment thread .npmrc

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

revert. legacy-peer-deps=true is not necessary

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.

Update to Angular 22

4 participants