Skip to content

build: update all non-major dependencies#46707

Closed
angular-robot wants to merge 2 commits intoangular:mainfrom
angular-robot:ng-renovate/all-non-major-dependencies
Closed

build: update all non-major dependencies#46707
angular-robot wants to merge 2 commits intoangular:mainfrom
angular-robot:ng-renovate/all-non-major-dependencies

Conversation

@angular-robot
Copy link
Contributor

@angular-robot angular-robot commented Jul 5, 2022

This PR contains the following updates:

Package Type Update Change
@types/babel__traverse dependencies minor 7.17.1 -> 7.18.0
@types/chrome dependencies patch ^0.0.190 -> ^0.0.193
@types/jquery devDependencies patch 3.5.1 -> 3.5.14
angular-route (source) dependencies patch 1.8.0 -> 1.8.3
io_bazel_rules_sass http_archive minor 1.50.0 -> 1.54.3
jasmine devDependencies minor ~4.2.0 -> ~4.3.0
jasmine-core devDependencies minor ~4.1.0 -> ~4.3.0
jasmine-core devDependencies minor ~4.2.0 -> ~4.3.0
karma dependencies minor ~6.3.0 -> ~6.4.0
karma devDependencies minor ~6.3.0 -> ~6.4.0
karma devDependencies minor ~6.3.2 -> ~6.4.0
karma-jasmine devDependencies minor ~5.0.0 -> ~5.1.0
rollup (source) dependencies minor ~2.75.0 -> ~2.77.0
ts-node (source) devDependencies minor ~10.8.0 -> ~10.9.0
tsec devDependencies patch 0.2.5 -> 0.2.6

Configuration

📅 Schedule: Branch creation - "after 10:00pm on monday,before 04:00am on tuesday" in timezone America/Tijuana, Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Renovate will not automatically rebase this PR, because other commits have been found.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, click this checkbox. ⚠ Warning: custom changes will be lost.

This PR has been generated by Renovate Bot.

@angular-robot angular-robot added action: review The PR is still awaiting reviews from at least one requested reviewer area: build & ci Related the build and CI infrastructure of the project target: patch This PR is targeted for the next patch release labels Jul 5, 2022
@ngbot ngbot bot modified the milestone: Backlog Jul 5, 2022
@pullapprove pullapprove bot requested a review from josephperrott July 5, 2022 05:09
@angular-robot angular-robot force-pushed the ng-renovate/all-non-major-dependencies branch 5 times, most recently from edc11e4 to 8888629 Compare July 11, 2022 21:04
@devversion devversion added state: blocked and removed action: review The PR is still awaiting reviews from at least one requested reviewer labels Jul 13, 2022
@devversion
Copy link
Member

Blocked on the karma-jasmine update and zone.js fix I sent separately..

@angular-robot angular-robot force-pushed the ng-renovate/all-non-major-dependencies branch 4 times, most recently from 6fc573b to 00d62f2 Compare July 15, 2022 10:37
@devversion
Copy link
Member

The karma jasmine fix landed, rebasing..

@angular-robot angular-robot force-pushed the ng-renovate/all-non-major-dependencies branch from 00d62f2 to ba3f934 Compare July 18, 2022 19:34
devversion added a commit to devversion/angular that referenced this pull request Jul 19, 2022
The source-map package is updated as part angular#46707, but the minor
version bump introduced breaking changes. This commit extracts the
update so that we can handle it separately accounting for the breaking
changes. Also removing the caret since this package apparently makes
breaking changes in minor segment bumps.
AndrewKushnir pushed a commit that referenced this pull request Jul 19, 2022
The source-map package is updated as part #46707, but the minor
version bump introduced breaking changes. This commit extracts the
update so that we can handle it separately accounting for the breaking
changes. Also removing the caret since this package apparently makes
breaking changes in minor segment bumps.

PR Close #46888
AndrewKushnir pushed a commit that referenced this pull request Jul 19, 2022
The source-map package is updated as part #46707, but the minor
version bump introduced breaking changes. This commit extracts the
update so that we can handle it separately accounting for the breaking
changes. Also removing the caret since this package apparently makes
breaking changes in minor segment bumps.

PR Close #46888
AndrewKushnir pushed a commit that referenced this pull request Jul 19, 2022
The source-map package is updated as part #46707, but the minor
version bump introduced breaking changes. This commit extracts the
update so that we can handle it separately accounting for the breaking
changes. Also removing the caret since this package apparently makes
breaking changes in minor segment bumps.

PR Close #46888
@angular-robot angular-robot force-pushed the ng-renovate/all-non-major-dependencies branch from ba3f934 to 48817fe Compare July 19, 2022 16:44
@josephperrott josephperrott requested review from devversion and removed request for josephperrott July 22, 2022 13:26
@pullapprove pullapprove bot requested a review from alxhub July 22, 2022 13:26
gkalpak added a commit to gkalpak/angular that referenced this pull request Aug 4, 2022
Since Renovate runs from the project directory, it cannot find the
`sync-deps` yarn script inside `aio/tools/examples/shared/`:
[example failure][1]

Update the post-upgrade task command to run inside that directory
instead of the project root.

[1]: angular#46707 (comment)
dylhunn pushed a commit that referenced this pull request Aug 4, 2022
Since Renovate runs from the project directory, it cannot find the
`sync-deps` yarn script inside `aio/tools/examples/shared/`:
[example failure][1]

Update the post-upgrade task command to run inside that directory
instead of the project root.

[1]: #46707 (comment)

PR Close #47040
dylhunn pushed a commit that referenced this pull request Aug 4, 2022
Since Renovate runs from the project directory, it cannot find the
`sync-deps` yarn script inside `aio/tools/examples/shared/`:
[example failure][1]

Update the post-upgrade task command to run inside that directory
instead of the project root.

[1]: #46707 (comment)

PR Close #47040
@angular-robot angular-robot force-pushed the ng-renovate/all-non-major-dependencies branch 2 times, most recently from 89a94e2 to 0630693 Compare August 4, 2022 21:35
@devversion
Copy link
Member

devversion commented Aug 5, 2022

Blocked on google/safevalues#190 / cl/465542098

devversion added a commit to devversion/angular that referenced this pull request Aug 5, 2022
Updates `safevalues` to unblock
angular#46707. As part of this commit
the necessary refactorings due to API changes are also made.
atscott pushed a commit that referenced this pull request Aug 5, 2022
…#47048)

Updates `safevalues` to unblock
#46707. As part of this commit
the necessary refactorings due to API changes are also made.

PR Close #47048
atscott pushed a commit that referenced this pull request Aug 5, 2022
…#47048)

Updates `safevalues` to unblock
#46707. As part of this commit
the necessary refactorings due to API changes are also made.

PR Close #47048
@angular-robot angular-robot force-pushed the ng-renovate/all-non-major-dependencies branch from 0630693 to 6f13674 Compare August 5, 2022 17:09
Copy link
Member

@devversion devversion left a comment

Choose a reason for hiding this comment

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

LGTM. Finally green after jumping around in so many places..

Reviewed-for: global-dev-infra-approvers

angular-robot and others added 2 commits August 6, 2022 08:16
See associated pull request for more information.
Yarn 1.x. is known to not prune nested unused node modules. This throws
off Bazel when the dependency tree changes but there are leftover unused
nested node module folders. This causes CI failures currently after the
dependency tree updates.

Invalidating the cache fixes this issue. Long-term a switch to Yarn
2.x+, pnpm will fix this.
@devversion devversion force-pushed the ng-renovate/all-non-major-dependencies branch from 6f13674 to 66695cc Compare August 6, 2022 08:19
@devversion devversion added action: merge The PR is ready for merge by the caretaker and removed state: blocked labels Aug 6, 2022
@devversion devversion removed request for alxhub and gkalpak August 6, 2022 08:19
@dylhunn
Copy link
Contributor

dylhunn commented Aug 8, 2022

This PR was merged into the repository by commit 2548214.

dylhunn pushed a commit that referenced this pull request Aug 8, 2022
See associated pull request for more information.

PR Close #46707
dylhunn pushed a commit that referenced this pull request Aug 8, 2022
Yarn 1.x. is known to not prune nested unused node modules. This throws
off Bazel when the dependency tree changes but there are leftover unused
nested node module folders. This causes CI failures currently after the
dependency tree updates.

Invalidating the cache fixes this issue. Long-term a switch to Yarn
2.x+, pnpm will fix this.

PR Close #46707
@dylhunn dylhunn closed this in ab9ce8a Aug 8, 2022
dylhunn pushed a commit that referenced this pull request Aug 8, 2022
Yarn 1.x. is known to not prune nested unused node modules. This throws
off Bazel when the dependency tree changes but there are leftover unused
nested node module folders. This causes CI failures currently after the
dependency tree updates.

Invalidating the cache fixes this issue. Long-term a switch to Yarn
2.x+, pnpm will fix this.

PR Close #46707
@angular-robot angular-robot deleted the ng-renovate/all-non-major-dependencies branch August 9, 2022 12:27
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 9, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants