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

use new artifacts-common library #13963

Merged
merged 3 commits into from
Nov 24, 2020

Conversation

shadargee1982
Copy link
Contributor

@shadargee1982 shadargee1982 commented Nov 23, 2020

Task name: NuGetAuthenticateV0

Description: This PR updates the version of azure-pipelines-tasks-artifacts-common. The updated common library has fixes to handle retry logic for common network errors.

Documentation changes required: N

Added unit tests: (Y/N)

Attached related issue: https://portal.microsofticm.com/imp/v3/incidents/details/214499609/home

Checklist:

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

@damccorm
Copy link

@microsoft/akvelon-build-task-team or @alex-peck any idea why this build is failing? As far as I can tell all the upgrade steps were followed

@alex-peck
Copy link
Contributor

@damccorm The TS version needs to be set in devDependencies.

@damccorm
Copy link

Good catch - thanks

@shadargee1982
Copy link
Contributor Author

@damccorm The TS version needs to be set in devDependencies.

That fixed it. Thanks.

@shadargee1982 shadargee1982 merged commit 220abbb into master Nov 24, 2020
@shadargee1982 shadargee1982 deleted the users/shrverm/update-nugetAuth-task branch November 24, 2020 23:52
@ritchxu
Copy link

ritchxu commented Dec 2, 2020

@shadargee1982 @damccorm @aasim I'm getting following error with version 0.179.0, would this PR be related?

##[error]File not found: 'D:\a\_tasks\NuGetAuthenticate_f5fd8599-ccfa-4d6e-b965-4d14bed7097b\0.179.0\bash.js'

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.

5 participants