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

Updated install instructions for new users/Windows #402

Merged
merged 3 commits into from
Feb 2, 2024

Conversation

beckyosb
Copy link
Contributor

@beckyosb beckyosb commented Feb 1, 2024

Fixes #401 by removing the invalid character & missing command that prevent the install command from running from the Windows command line.

@CLAassistant
Copy link

CLAassistant commented Feb 1, 2024

CLA assistant check
All committers have signed the CLA.

@hkamran80
Copy link

Alternatively, you could change the original command to use ./ instead of the pwd command. Both do the same thing.

@beckyosb
Copy link
Contributor Author

beckyosb commented Feb 1, 2024

I assumed, I just opted on the side of being conservative! Either/or.

@beckyosb
Copy link
Contributor Author

beckyosb commented Feb 1, 2024

I simplified the install guide per @hkamran80 & added a link to the Docker install guide for users who might not be familiar.

@beckyosb beckyosb changed the title Updated README.md with differentiated instructions for Windows Updated install instructions for new users/Windows Feb 1, 2024
Made redundant by projects
@AnalogJ
Copy link
Member

AnalogJ commented Feb 2, 2024

appreciate the doc updates @beckyosb these are super helpful!

Merging 🥳

@AnalogJ AnalogJ merged commit b40b0fe into fastenhealth:main Feb 2, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug]: Install instructions produce an error on Windows
4 participants