Skip to content

Conversation

@johnterickson
Copy link
Contributor

@johnterickson johnterickson commented Sep 1, 2022

Bug

Fixes: NuGet/Home#11793

Regression? No

Description

Plugin timeout defaults are increased from 5 seconds to 30 seconds based on discussion in issue NuGet/Home#11793 and NuGet/Home#8528 (comment)

PR Checklist

  • PR has a meaningful title

  • PR has a linked issue.

  • Described changes

  • Tests

    • Automated tests added
    • OR
    • Test exception
    • OR
    • N/A
  • Documentation

    • Documentation PR or issue filled
    • OR
    • N/A

@johnterickson johnterickson requested a review from a team as a code owner September 1, 2022 16:33
@johnterickson
Copy link
Contributor Author

If this looks good, we should also apply to servicing branches.

nkolev92
nkolev92 previously approved these changes Sep 1, 2022
@nkolev92
Copy link
Member

nkolev92 commented Sep 1, 2022

Thanks for creating this PR @johnterickson

It'd be great if you can reach out to Arturo for servicing considerations.
cc @aortiz-msft

@johnterickson
Copy link
Contributor Author

@nkolev92 Could you reapprove now that I fixed the test regex?

@johnterickson
Copy link
Contributor Author

@nkolev92 Looks ready to merge. Anything else?

@nkolev92 nkolev92 merged commit 07eaa53 into NuGet:dev Sep 6, 2022
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.

[DCR]: Plugin timeout defaults should be increased

2 participants