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

Update the PR spec template with information about how to get help from the ARM API reviewer in the absence of the PR assignee #27362

Closed
wants to merge 1 commit into from

Conversation

rkmanda
Copy link
Member

@rkmanda rkmanda commented Jan 12, 2024

Going forward, the PR assignee for each control plane PR will be left empty. This is an intentional change to streamline the collaboration between the PR author and the ARM reviewers. We now ask the authors to reserve an ARM office hours slot to discuss any questions they might have about their PR or if they need to get unblocked.

Relevant issue:

Copy link

openapi-pipeline-app bot commented Jan 12, 2024

Next Steps to Merge

✔️ All automated merging requirements have been met! Refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

Copy link

openapi-pipeline-app bot commented Jan 12, 2024

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

Copy link

openapi-pipeline-app bot commented Jan 12, 2024

Swagger Generation Artifacts

️️✔️ApiDocPreview succeeded [Detail] [Expand]
️️✔️SDK Breaking Change Tracking succeeded [Detail] [Expand]

Breaking Changes Tracking

Posted by Swagger Pipeline | How to fix these errors?

Copy link

PR validation pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@rkmanda rkmanda changed the title Update the PR spec template about how to get help from the ARM API reviewer in the absence of the PR assignee Update the PR spec template with information about how to get help from the ARM API reviewer in the absence of the PR assignee Jan 12, 2024
@konrad-jamrozik
Copy link

@rkmanda let's not add such large amount of text to the PR description: we were constantly running into the problem of having too much text which nobody reads, hence we are quite strict on the amount of text we are willing to put into a PR description. I will create a separate PR to do this update and offload most of this information into a dedicated aka.ms link in aka.ms/azsdkdocs. I will use this PR of yours for info what to write. Thx!

@konrad-jamrozik
Copy link

konrad-jamrozik commented Jan 17, 2024

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.

3 participants