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

Added example on how to scale Azure Pipelines #84155

Merged
merged 3 commits into from Jan 28, 2022
Merged

Added example on how to scale Azure Pipelines #84155

merged 3 commits into from Jan 28, 2022

Conversation

SebastianSchuetze
Copy link
Contributor

Hope the info is enough. I added also a blog post link

Hope the info is enough. I added also a blog post link
@PRMerger3
Copy link
Contributor

@SebastianSchuetze : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@ktoliver
Copy link
Contributor

#label:"aq-pr-triaged"

@PRMerger8 PRMerger8 added the aq-pr-triaged tracking label for the PR review team label Nov 19, 2021
@SebastianSchuetze
Copy link
Contributor Author

Any progress on this? I think this example can benefit many users already trying to adopt.

@ktoliver
Copy link
Contributor

ktoliver commented Dec 3, 2021

@craigshoemaker for review. Thanks!

Copy link
Contributor

@craigshoemaker craigshoemaker left a comment

Choose a reason for hiding this comment

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

Thanks for this PR, @SebastianSchuetze!

I suggested some changes to the wording, and removed a link to a personal blog.

articles/container-apps/scale-app.md Outdated Show resolved Hide resolved
articles/container-apps/scale-app.md Outdated Show resolved Hide resolved
Co-authored-by: Craig Shoemaker <craig@craigshoemaker.net>
@PRMerger8
Copy link
Contributor

@SebastianSchuetze : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Co-authored-by: Craig Shoemaker <craig@craigshoemaker.net>
@PRMerger5
Copy link
Contributor

@SebastianSchuetze : Thanks for your contribution! The author(s) have been notified to review your proposed change.

@SebastianSchuetze
Copy link
Contributor Author

@craigshoemaker added you suggestions!

@SebastianSchuetze
Copy link
Contributor Author

@craigshoemaker is this now fine? If yes can this be merged now?

@SebastianSchuetze
Copy link
Contributor Author

@ktoliver @craigshoemaker worked on the requested changes. Can we merge it?

@ktoliver
Copy link
Contributor

I'm going to merge this because the author's feedback was incorporated and it LGTM. Thanks, all.

@ktoliver ktoliver merged commit ed6c790 into MicrosoftDocs:master Jan 28, 2022
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.

None yet

7 participants