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.

Sign up for GitHub

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

Jump to bottom

Simplifying the documentation #87

Merged

sushilkm merged 7 commits into Azure:main from sushilkm:suskuma/doc-fixes

Oct 17, 2022

Contributor

sushilkm commented Oct 4, 2022 •

edited

Loading

Fixes #83

sushilkm requested review from senthilkungumaraj, karthick-rn and mandarinamdar

October 4, 2022 22:31

sushilkm force-pushed the suskuma/doc-fixes branch from 1ac71d1 to 44a86b6 Compare

October 7, 2022 16:49

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Outdated Show resolved Hide resolved

karthick-rn reviewed

View reviewed changes

deploy/README.md Show resolved Hide resolved

Contributor

karthick-rn commented Oct 11, 2022 •

edited

Loading

Please review only the last commit titled "Updated documentation", the other two commits are already covered in #86

@sushilkm, As we merged #86 we can remove this line to avoid confusion.

sushilkm added 3 commits

October 11, 2022 12:49


          Updated documentation

4eb73ee


          Simplify the documentation


          Fixed as per comments

d60823f

sushilkm force-pushed the suskuma/doc-fixes branch from 90b8dc9 to d60823f Compare

October 11, 2022 20:03


          Added RBAC requirement section

6ee118e

senthilkungumaraj approved these changes

View reviewed changes

docs/Troubleshooting.md Outdated Show resolved Hide resolved

deploy/bring-your-own-ai-model.md Outdated Show resolved Hide resolved

azure-pipelines/PR-pipeline.yml Outdated Show resolved Hide resolved

README.md Outdated Show resolved Hide resolved

deploy/bring-your-own-ai-model.md Outdated Show resolved Hide resolved


          Comments addressed

df088fc

mandarinamdar requested changes

View reviewed changes

Member

mandarinamdar left a comment

I have left the comments in ongoing conversations


          Addressed comments

d843d55

sushilkm mentioned this pull request

Add more information in Troubleshooting .md #91

Open

karthick-rn approved these changes

View reviewed changes

mandarinamdar reviewed

View reviewed changes

Member

mandarinamdar left a comment

Lets create issue for structuring the Deploy folder better in a separate PR. rest looks good. thanks for addressing the comments

mandarinamdar approved these changes

View reviewed changes


          Addressed comments

67f1fa1

sushilkm merged commit 5df7dc9 into Azure:main

sushilkm deleted the suskuma/doc-fixes branch

October 17, 2022 19:32

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

Labels

None yet