Skip to content

build: update jasmine dependencies to v7 (main) - #33868

Open
angular-robot wants to merge 1 commit into
angular:mainfrom
angular-robot:ng-renovate/main-major-jasmine-dependencies
Open

build: update jasmine dependencies to v7 (main)#33868
angular-robot wants to merge 1 commit into
angular:mainfrom
angular-robot:ng-renovate/main-major-jasmine-dependencies

Conversation

@angular-robot

Copy link
Copy Markdown
Contributor

This PR contains the following updates:

Package Change Age Adoption Passing Confidence
jasmine (source) ~6.3.0~7.0.0 age adoption passing confidence
jasmine (source) ^6.0.0^7.0.0 age adoption passing confidence
jasmine-core (source) ~6.3.0~7.0.0 age adoption passing confidence

  • If you want to rebase/retry this PR, check this box

Release Notes

jasmine/jasmine-npm (jasmine)

v7.0.0

Compare Source

Please see the release notes.

jasmine/jasmine (jasmine-core)

v7.0.1

Compare Source

Please see the release notes.

v7.0.0

Compare Source

Please see the release notes.

See associated pull request for more information.
@angular-robot angular-robot added action: merge The PR is ready for merge by the caretaker area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only] labels Aug 17, 2026

@gemini-code-assist gemini-code-assist Bot 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.

Code Review

This pull request upgrades jasmine and jasmine-core to version 7.0.0 across several package.json files. The feedback points out that the corresponding @types/jasmine dependencies should also be updated to ~7.0.0 to match the new major version and prevent potential TypeScript compilation errors.

Comment thread package.json
"husky": "9.1.7",
"jasmine": "~6.3.0",
"jasmine-core": "~6.3.0",
"jasmine": "~7.0.0",

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

When upgrading jasmine and jasmine-core to v7, the @types/jasmine dependency (currently at ~6.0.0 on line 81) should also be updated to ~7.0.0 to align type definitions with the new major version and avoid potential TypeScript compilation errors.

"@types/node": "^20.17.19",
"@types/jasmine": "~6.0.0",
"jasmine": "^6.0.0"
"jasmine": "^7.0.0"

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

When upgrading jasmine to ^7.0.0, the @types/jasmine dependency (currently at ~6.0.0 on line 22) should also be updated to ~7.0.0 to align type definitions with the new major version and avoid potential TypeScript compilation errors.

"@types/node": "^20.17.19",
"@types/jasmine": "~6.0.0",
"jasmine": "~6.3.0",
"jasmine": "~7.0.0",

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

When upgrading jasmine to ~7.0.0, the @types/jasmine dependency (currently at ~6.0.0 on line 21) should also be updated to ~7.0.0 to align type definitions with the new major version and avoid potential TypeScript compilation errors.

"express": "^5.1.0",
"istanbul-lib-instrument": "^6.0.3",
"jasmine-core": "~6.3.0",
"jasmine-core": "~7.0.0",

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

When upgrading jasmine-core to ~7.0.0, the @types/jasmine dependency (currently at ~6.0.0 on line 7) should also be updated to ~7.0.0 to align type definitions with the new major version and avoid potential TypeScript compilation errors.

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 area: build & ci Related the build and CI infrastructure of the project target: automation This PR is targeted to only merge into the branch defined in Github [bot use only]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant