Skip to content

build: add package metadata to @angular/ssr package.json#33195

Merged
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:ssr-fix
May 13, 2026
Merged

build: add package metadata to @angular/ssr package.json#33195
alan-agius4 merged 1 commit into
angular:mainfrom
alan-agius4:ssr-fix

Conversation

@alan-agius4
Copy link
Copy Markdown
Collaborator

Unlike ng_package doesn't add these fields.

Unlike ng_package doesn't add these fields.
@alan-agius4 alan-agius4 requested a review from dgp1130 May 13, 2026 19:18
@angular-robot angular-robot Bot added the area: build & ci Related the build and CI infrastructure of the project label May 13, 2026
@alan-agius4 alan-agius4 added target: rc This PR is targeted for the next release-candidate action: merge The PR is ready for merge by the caretaker and removed area: build & ci Related the build and CI infrastructure of the project labels May 13, 2026
Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the package.json for the @angular/ssr package by adding metadata fields including repository, author, license, and issue tracker information. The reviewer suggested adding a directory property to the repository object to correctly link to the package's source within the monorepo.

Comment on lines +11 to +14
"repository": {
"type": "git",
"url": "git+https://github.com/angular/angular-cli.git"
},
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

medium

In a monorepo, it is recommended to include the directory property within the repository object for sub-packages. This allows tools and registries (like npmjs.com) to correctly link to the source code of the specific package.

Suggested change
"repository": {
"type": "git",
"url": "git+https://github.com/angular/angular-cli.git"
},
"repository": {
"type": "git",
"url": "git+https://github.com/angular/angular-cli.git",
"directory": "packages/angular/ssr"
},

@alan-agius4 alan-agius4 merged commit f0a64fe into angular:main May 13, 2026
37 checks passed
@alan-agius4 alan-agius4 deleted the ssr-fix branch May 13, 2026 19:50
@alan-agius4
Copy link
Copy Markdown
Collaborator Author

This PR was merged into the repository. The changes were merged into the following branches:

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

Labels

action: merge The PR is ready for merge by the caretaker target: rc This PR is targeted for the next release-candidate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants