Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[AppCenterDistributeV1] Update the Nock package dependency #15371

Merged
merged 4 commits into from
Oct 10, 2021

Conversation

DenisRumyantsev
Copy link
Contributor

@DenisRumyantsev DenisRumyantsev commented Oct 5, 2021

Task name: AppCenterDistributeV1

Description:

We need to update the nock package dependency for the AppCenterDistributeV1 task to fix the lodash dependabot alert.
Nock version was updated from 9.0.2 to 10.0.6.
Lodash version was automatically updated from 4.9.0 to 4.17.21 (currently latest).
There are breaking changes between 10.0.6 and 11.3.2. You can see breaking changes in the release notes.
So we have to use the 10.0.6 Nock version.

Documentation changes required: (Y/N)

Added unit tests: (Y/N)

Attached related issue: (Y/N)

Checklist:

  • Task version was bumped - please check instruction how to do it
  • Checked that applied changes work as expected

AppCenterDistributeV1 task with the updated nock package dependency was successfully built, deployed to the Azure DevOps organization, and added to a pipeline. I did not run this pipeline since this task requires creating an application and App Center service connection as server endpoint input. It is enough for testing since the updated dependency is related to unit-tests which work correctly with the 10.0.6 Nock version.

@DenisRumyantsev DenisRumyantsev changed the title AppCenterDistributeV1 nock 9.0.2 -> 10.0.6 Update the Nock package dependency for the AppCenterDistributeV1 task Oct 5, 2021
@DenisRumyantsev DenisRumyantsev requested a review from a team October 6, 2021 09:01
@DenisRumyantsev DenisRumyantsev changed the title Update the Nock package dependency for the AppCenterDistributeV1 task [AppCenterDistributeV1] Update the Nock package dependency Oct 6, 2021
@DenisRumyantsev DenisRumyantsev merged commit 7566155 into master Oct 10, 2021
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.

4 participants