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

Make a lot of experimental things release #969

Merged
merged 2 commits into from May 18, 2021

Conversation

JohnMcPMS
Copy link
Member

@JohnMcPMS JohnMcPMS commented May 17, 2021

Change

Make list, upgrade, uninstall, and export commands no longer experimental. Also make the REST source implementation not experimental, as well as making Delivery Optimization the default downloader.

Microsoft Reviewers: Open in CodeFlow

@JohnMcPMS JohnMcPMS requested a review from a team as a code owner May 17, 2021 22:41
@@ -11,15 +11,12 @@ public class ImportCommand : BaseCommand
[SetUp]
public void Setup()
{
InitializeAllFeatures(false);
ConfigureFeature("export", true);
CleanupTestExe();
}

[TearDown]
public void TearDown()
Copy link
Contributor

Choose a reason for hiding this comment

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

public void TearDown()

Should we just delete the tear down?

Copy link
Contributor

Choose a reason for hiding this comment

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

Applies to the other 2 tests' setup and teardown if you decide to do so

Copy link
Contributor

@yao-msft yao-msft left a comment

Choose a reason for hiding this comment

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

:shipit:

// While DO still requires an explicit opt-in, we will let failures through.
// When DO becomes the default, we may choose to catch exceptions and fall back to WinINet below.
// If DO becomes an issue, we may choose to catch exceptions and fall back to WinINet below.
// We would need to be careful not to bypass metered networks or other reasons that might
Copy link
Contributor

Choose a reason for hiding this comment

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

other reasons

Curious what the other reasons could be?
Is metered networks the reason we do not fall back to wininet for now?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants