Skip to content
This repository has been archived by the owner on Oct 5, 2023. It is now read-only.

deploy individual initiatives #179

Merged
merged 16 commits into from
Jun 20, 2023
Merged

deploy individual initiatives #179

merged 16 commits into from
Jun 20, 2023

Conversation

arjenhuitema
Copy link
Collaborator

Added documentation to deploy individual initiatives.

paulgrimley
paulgrimley previously approved these changes Jun 14, 2023
Copy link
Contributor

@paulgrimley paulgrimley left a comment

Choose a reason for hiding this comment

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

Approved

@paulgrimley paulgrimley dismissed their stale review June 14, 2023 10:11

approved to early

Copy link
Contributor

@paulgrimley paulgrimley left a comment

Choose a reason for hiding this comment

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

Requested some changes so we make sure Service Health is more prominent and also wanted some clarity on a couple of things

docs/wiki/Deploy-individual-Policy-Initiatives.md Outdated Show resolved Hide resolved
docs/wiki/Deploy-individual-Policy-Initiatives.md Outdated Show resolved Hide resolved
docs/wiki/Deploy-individual-Policy-Initiatives.md Outdated Show resolved Hide resolved
docs/wiki/Introduction-to-deploying-ALZ-Monitor.md Outdated Show resolved Hide resolved
docs/wiki/_Sidebar.md Outdated Show resolved Hide resolved
arjenhuitema and others added 4 commits June 14, 2023 07:56
Co-authored-by: Paul Grimley <25264573+paulgrimley@users.noreply.github.com>
Co-authored-by: Paul Grimley <25264573+paulgrimley@users.noreply.github.com>
paulgrimley
paulgrimley previously approved these changes Jun 19, 2023
Copy link
Contributor

@paulgrimley paulgrimley left a comment

Choose a reason for hiding this comment

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

Thanks @arjenhuitema LGTM, @jfaurskov can you take a look?

docs/wiki/_Sidebar.md Outdated Show resolved Hide resolved
docs/wiki/_Sidebar.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@jfaurskov jfaurskov left a comment

Choose a reason for hiding this comment

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

Excellent work @arjenhuitema just a few suggestions. :-)

Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
arjenhuitema and others added 3 commits June 20, 2023 09:14
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
Co-authored-by: Jan Faurskov <22591930+jfaurskov@users.noreply.github.com>
@arjenhuitema
Copy link
Collaborator Author

@jfaurskov great suggestions. Committed all.

Copy link
Collaborator

@jfaurskov jfaurskov left a comment

Choose a reason for hiding this comment

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

LGTM

@jfaurskov jfaurskov merged commit c6622e3 into Azure:main Jun 20, 2023
@jfaurskov jfaurskov removed the request for review from paulgrimley June 20, 2023 14:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants