-
Notifications
You must be signed in to change notification settings - Fork 202
Migrate SkipDefaultCheckout to SkipCheckoutNone #4042
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
Conversation
@@ -8,13 +8,17 @@ parameters: | |||
- Name: $(Build.Repository.Name) | |||
Commitish: $(Build.SourceVersion) | |||
WorkingDirectory: $(System.DefaultWorkingDirectory) | |||
# NOTE: SkipDefaultCheckout is being deprecated in favor of SkipCheckoutNone |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm keeping both to make it easier to do PRs one by one in the other repositories. After I'm done, I'll remove this parameter.
The following pipelines have been queued for testing: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I originally was like
eugh
to the name. Then I read the variable, and I can't really hate anymore. It's definitely clearer now than it was.
Hello @azure-sdk! Because this pull request has the p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (
|
Attempting to find a name for this parameter that causes less confusion