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.

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

Documentation Issue #1674

Closed
AjinkyaRaghuwanshi opened this issue May 21, 2021 · 3 comments · Fixed by #1678 · May be fixed by #1682
Closed

Documentation Issue #1674

AjinkyaRaghuwanshi opened this issue May 21, 2021 · 3 comments · Fixed by #1678 · May be fixed by #1682
Assignees
@acomodi
Labels
done

Comments

@AjinkyaRaghuwanshi
Copy link

The step 6 in the Quick start guide should be done after completing the step 7 and 8. Otherwise it throws an error.
Also the documentation does not specifically provide the directory the command should be ran in for ex: after the step 9 we will have to move to a different directory for step 10 and then follow the C++ commands step, so please take care of these issue,
@issuelabeler issuelabeler bot added the done label May 21, 2021
@mithro
Copy link
Contributor

mithro commented May 24, 2021

@acomodi - Who is the correct person to fix this?

@acomodi
Copy link
Contributor

acomodi commented May 24, 2021

I can fix this, it is quite straightforward. Long term we might use tuttest to verify the readme commands are working properly

@acomodi acomodi self-assigned this May 24, 2021
@acomodi acomodi mentioned this issue May 25, 2021
Merged
@benglines benglines mentioned this issue Jun 3, 2021
Open
@benglines
Copy link

I saw that the wording was fixed to show that for Step 10 you need to be in a different directory, but I didn't see any changes to the ordering of the quick start guide steps, so I linked a pull request that puts step 6 after steps 7 and 8

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

@acomodi acomodi

Labels
done
Projects
None yet
Milestone
No milestone
4 participants
@mithro @acomodi @benglines @AjinkyaRaghuwanshi