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

Updates to Accelerator Documentation #614

Merged
merged 3 commits into from
Aug 18, 2023

Conversation

oZakari
Copy link
Contributor

@oZakari oZakari commented Aug 16, 2023

Overview/Summary

AB#30070
AB#30071

Add clarity to Accelerator steps and transition to clone approach for remote repository.

This PR fixes/adds/changes/removes

  1. Adds clarity to Accelerator documentation steps
  2. Transitions to clone approach for remote repository

Breaking Changes

  1. None, documentation only

Testing Evidence

N/A

As part of this Pull Request I have

jtracey93
jtracey93 previously approved these changes Aug 16, 2023
Copy link

@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.

@oZakari one comment from me, there was an issue with my git config re main and master - could you add an optional command to set to main as part of setup

@oZakari
Copy link
Contributor Author

oZakari commented Aug 18, 2023

@oZakari one comment from me, there was an issue with my git config re main and master - could you add an optional command to set to main as part of setup

Thanks for looking it over @paulgrimley! I've actually removed that branch rename step as I've transitioned away from initializing the repo locally and instead the steps are to just clone the repo down from GitHub/ADO so it simplifies the process quite a bit and you don't have to worry about setting of your local git installation.

image

@microsoft-github-policy-service microsoft-github-policy-service bot added Needs: Attention 👋 Needs attention from the maintainers and removed Needs: Author Feedback labels Aug 18, 2023
@oZakari oZakari merged commit 9756593 into Azure:main Aug 18, 2023
4 checks passed
@oZakari oZakari deleted the update-accelerator-doc branch November 21, 2023 20:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs: Attention 👋 Needs attention from the maintainers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants