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

Fix platform-specific setup instructions #484

Open
ndporter opened this issue Jul 29, 2020 · 1 comment
Open

Fix platform-specific setup instructions #484

ndporter opened this issue Jul 29, 2020 · 1 comment

Comments

@ndporter
Copy link

ndporter commented Jul 29, 2020

The setup instructions include the following line:

If it’s not unzipped yet, double-click on it to unzip it. You should end up with a new folder called data.

Double-click to unzip is default behavior on Mac but not Windows (official Windows directions). I assume there's some variability on Linux. Directions should not be platform-specific for lessons designed for multiple platforms.

For consistency with the abbreviated nature of the remainder of the instructions, I suggest changing to:

If it’s not unzipped yet, unzip it. There should now be a folder called data in the r-novice-inflammation folder.

If the remainder of the instructions were more detailed, it might make sense to link to platform-specific instructions for true beginners there (that may not know how to unzip) but in that case, all three platforms would need to be represented.

@HaoZeke
Copy link
Member

HaoZeke commented Aug 5, 2020

Excellent catch! Could you open a PR with the minimal instruction set? I'm reasonably certain handling three platforms for unzipping in this lesson is a bit overkill. Thanks and sorry for my late response

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

No branches or pull requests

2 participants