Skip to content

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Apr 18, 2022

WhiteSource Renovate

This PR contains the following updates:

Update Change
lockFileMaintenance All locks refreshed

🔧 This Pull Request updates lock files to use the latest dependency versions.


Configuration

📅 Schedule: "before 5am on monday" in timezone America/Tijuana.

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

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

👻 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.

This PR has been generated by WhiteSource Renovate. View repository job log here.

@renovate renovate bot added action: merge The PR is ready for merge by the caretaker target: minor This PR is targeted for the next minor release labels Apr 18, 2022
@renovate renovate bot force-pushed the renovate/lock-file-maintenance branch 5 times, most recently from 84e06ef to 979b498 Compare April 22, 2022 15:35
@clydin clydin added this to the v14 milestone Apr 27, 2022
@renovate renovate bot force-pushed the renovate/lock-file-maintenance branch 4 times, most recently from 58e4944 to 5658beb Compare May 3, 2022 21:50
@clydin clydin requested a review from dgp1130 May 4, 2022 16:26
dgp1130
dgp1130 previously approved these changes May 4, 2022
Copy link
Collaborator

@dgp1130 dgp1130 left a comment

Choose a reason for hiding this comment

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

Nit: Should the lockfile update commit come after the other commits? That way CI should pass at each of these commits if something got rolled back.

copy[key] = deepCopy(valueCasted[key]);
}
valueCasted[copySymbol] = undefined;
delete valueCasted[copySymbol];
Copy link
Collaborator

Choose a reason for hiding this comment

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

Interesting, I was always unclear about the difference between delete and = undefined. Defined undefined properties strike again!

clydin and others added 3 commits May 4, 2022 12:42
…operty after use

The `deepCopy` utility function previously set its internal tracking symbol property to `undefined` after completion.
However, this still caused the property to exist with a value of `undefined`. Jasmine 4.1 now checks for symbols with
its `toEqual` expectation which resulted in failing tests. The internal tracking symbol property is now removed instead
via `delete`.
@clydin clydin force-pushed the renovate/lock-file-maintenance branch from eae622f to 771f635 Compare May 4, 2022 16:43
@clydin clydin requested a review from dgp1130 May 4, 2022 16:44
@dgp1130 dgp1130 merged commit 932cb9f into main May 4, 2022
@clydin clydin deleted the renovate/lock-file-maintenance branch May 4, 2022 17:26
@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 Jun 4, 2022
@alan-agius4 alan-agius4 changed the title build: lock file maintenance build: lock file maintenance [er] Jul 4, 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 target: minor This PR is targeted for the next minor release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants