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

Add template for insertion creation and use for pr val #55094

Merged
merged 27 commits into from
Jul 29, 2021

Conversation

dibarbet
Copy link
Member

@dibarbet dibarbet commented Jul 23, 2021

@dibarbet dibarbet force-pushed the dev/dibarbet/insertion_template branch from 6d35f42 to 8f85797 Compare July 23, 2021 22:50
@dibarbet dibarbet force-pushed the dev/dibarbet/insertion_template branch from daf7b75 to 55f4995 Compare July 24, 2021 00:50
@dibarbet dibarbet force-pushed the dev/dibarbet/insertion_template branch from 55f4995 to 112f8b8 Compare July 24, 2021 00:53
@dibarbet dibarbet force-pushed the dev/dibarbet/insertion_template branch from 218c2d1 to ea39faf Compare July 26, 2021 20:26
@dibarbet dibarbet force-pushed the dev/dibarbet/insertion_template branch from a3f7f7c to 59ce49a Compare July 26, 2021 23:31
@dibarbet dibarbet marked this pull request as ready for review July 28, 2021 00:30
@dibarbet dibarbet requested a review from a team as a code owner July 28, 2021 00:30
@dibarbet dibarbet requested a review from JoeRobich July 28, 2021 00:30
Copy link
Member

@JoeRobich JoeRobich left a comment

Choose a reason for hiding this comment

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

:shipit:

@dibarbet dibarbet enabled auto-merge July 28, 2021 00:37
eng/pipelines/insert.yml Show resolved Hide resolved
eng/pipelines/insert.yml Show resolved Hide resolved
eng/pipelines/insert.yml Show resolved Hide resolved
@dibarbet dibarbet disabled auto-merge July 28, 2021 01:04
@@ -0,0 +1,169 @@
parameters:
# These are actually a booleans but must be defined as string.
Copy link
Contributor

Choose a reason for hiding this comment

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

nit: "These are actually booleans"

@dibarbet dibarbet enabled auto-merge July 28, 2021 21:59
eng/pipelines/insert.yml Outdated Show resolved Hide resolved
eng/pipelines/insert.yml Show resolved Hide resolved
dibarbet and others added 2 commits July 28, 2021 15:56
Co-authored-by: Joey Robichaud <jorobich@microsoft.com>
Co-authored-by: Joey Robichaud <jorobich@microsoft.com>
@dibarbet dibarbet merged commit fd67237 into main Jul 29, 2021
@ghost ghost added this to the Next milestone Jul 29, 2021
@dibarbet dibarbet deleted the dev/dibarbet/insertion_template branch July 29, 2021 01:08
@dibarbet dibarbet modified the milestones: Next, 17.0.P4 Aug 31, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants