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

Enable menu to select platform #201

Closed
wants to merge 3 commits into from
Closed

Conversation

ErinBecker
Copy link
Contributor

Partial fix for #186

I've added the code to get the menu to work (I think), but as far as I can tell, the content for the section "Uploading Data to your Virtual Machine" for Unix is missing. This PR should NOT be merged without adding that content first. I haven't used scp in years. Anyone who can help add this content?

Partial fix for #186

I've added the code to get the menu to work (I think), but as far as I can tell, the content for the section "Uploading Data to your Virtual Machine" for Unix is missing. This PR should **NOT** be merged without adding that content first. I haven't used scp in years. Anyone who can help add this content?
@ErinBecker
Copy link
Contributor Author

Maybe @ACharbonneau or @taylorreiter could point me towards an older version of these instructions? I can type up the actual changes if someone can point me in the right direction. 🙂

@ACharbonneau
Copy link
Contributor

I think this is from a page originally called something like 'data roundtripping'. I'll see if I can find it

@ACharbonneau
Copy link
Contributor

and I can write/re-write the scp part

@ACharbonneau
Copy link
Contributor

If this came from the cloud-genomics lesson, it would be based on this: https://github.com/datacarpentry/cloud-genomics/blob/527ab4b3172cbb417db88ba51c612f803ce027e9/_episodes/04-data-roundtripping.md

@ErinBecker
Copy link
Contributor Author

ErinBecker commented May 7, 2019

Thanks @ACharbonneau for tracking that down! If you were ok with writing/rewriting the scp section, that would be fantastic. But I'm also available to help out if needed. Please just let me know. I'll take my cue from who you decide to assign this issue to. :-)

Extra fixes to complete #201
- fixed selection script
- moved words around so scp is under UNIX
@ACharbonneau ACharbonneau mentioned this pull request May 13, 2019
@ACharbonneau
Copy link
Contributor

ugh. I tried to just edit this PR, but it made a new one. Editing must be locked :(
I did use this PR as the base for #215 though, so you can safely merge 215 rather than this one and still get all Erin's changes.

@ACharbonneau
Copy link
Contributor

Incidentally, I also don't have privileges to assign myself to PRs in this repo, so sorry if it looked like I just disappeared :/

@ErinBecker
Copy link
Contributor Author

Replaced by #215.

@ErinBecker ErinBecker closed this May 14, 2019
@ErinBecker ErinBecker deleted the Enable-platform-selection branch May 14, 2019 19:00
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.

2 participants