Skip to content

Add basic descriptions to YAML pipelines - #200

Merged
Eugene Fedorenko (eedorenko) merged 3 commits into
microsoft:masterfrom
tcare:yaml-desc
Feb 21, 2020
Merged

Add basic descriptions to YAML pipelines#200
Eugene Fedorenko (eedorenko) merged 3 commits into
microsoft:masterfrom
tcare:yaml-desc

Conversation

@tcare

@tcare Tom Care (tcare) commented Feb 20, 2020

Copy link
Copy Markdown
Contributor

Added some basic descriptions and did some drive-by whitespace cleanup.

jotaylo
jotaylo reviewed Feb 21, 2020
View reviewed changes
Comment thread .pipelines/azdo-base-pipeline.yml Outdated
jotaylo
jotaylo reviewed Feb 21, 2020
View reviewed changes
Comment thread .pipelines/azdo-pr-build-train.yml Outdated
jotaylo
jotaylo reviewed Feb 21, 2020
View reviewed changes
Comment thread .pipelines/diabetes_regression-ci-build-train.yml Outdated
jotaylo
jotaylo reviewed Feb 21, 2020
View reviewed changes
Comment thread .pipelines/diabetes_regression-ci-build-train.yml Outdated
jotaylo
jotaylo reviewed Feb 21, 2020
View reviewed changes
Comment thread .pipelines/diabetes_regression-ci-build-train.yml Outdated
@jotaylo

jotaylo commented Feb 21, 2020

Copy link
Copy Markdown
Contributor

Can you add comments for these

Refers to: .pipelines/diabetes_regression-variables.yml:54 in 9e60ab4. [](commit_id = 9e60ab4, deletion_comment = False)

@tcare

Tom Care (tcare) commented Feb 21, 2020

Copy link
Copy Markdown
Contributor Author

Can you add comments for these

Refers to: .pipelines/diabetes_regression-variables.yml:54 in 9e60ab4. [](commit_id = 9e60ab4, deletion_comment = False)

Spoke offline, there needs to be a larger cleanup here and is out of scope for this PR.

jotaylo
jotaylo approved these changes Feb 21, 2020
View reviewed changes

@jotaylo jotaylo left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

:shipit:

@eedorenko
Eugene Fedorenko (eedorenko) merged commit eb3dc02 into microsoft:master Feb 21, 2020
@tcare

Tom Care (tcare) commented Feb 21, 2020

Copy link
Copy Markdown
Contributor Author

Oh, I was going to merge, but thanks :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

2 more reviewers
@jotaylo jotaylo jotaylo approved these changes
@eedorenko Eugene Fedorenko (eedorenko) eedorenko approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tcare @jotaylo @eedorenko