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

adding content to Tutorial 00 related to #72 #75

Merged
merged 3 commits into from
Nov 15, 2021
Merged

Conversation

virdi
Copy link
Member

@virdi virdi commented Nov 14, 2021

This updates Tutorial 00, as discussed in #72

@jules32
Copy link
Contributor

jules32 commented Nov 14, 2021

This looks good @virdi! I really like the "Steps" and "Discussion" sections, that is a great idea.

One question: with the Git plug-in, will they be prompted to configure their username and password, and if so do we need to cover that? Or is that somehow automatic?

@jules32
Copy link
Contributor

jules32 commented Nov 14, 2021

I'll let Luis merge after he has a look/adds updates

@virdi
Copy link
Member Author

virdi commented Nov 14, 2021

This looks good @virdi! I really like the "Steps" and "Discussion" sections, that is a great idea.

One question: with the Git plug-in, will they be prompted to configure their username and password, and if so do we need to cover that? Or is that somehow automatic?

For clone step, it won't prompt for github credentials. When they modify files in their cloned copy (fork), they will be prompted. Do we want to cover setting up credentials in Tutorial 00? If so, do we go the "access token" route and further, do we do it via command line?

I can include that if you want, and/or redirect them to the clinic chapter for self-guided set up. If we have time left, we can go over that too. Thoughts @betolink @jules32 ?

@virdi
Copy link
Member Author

virdi commented Nov 15, 2021

Just pushed some more minor edits (plus screenshots)

@jules32 jules32 merged commit 6fc0b40 into main Nov 15, 2021
@jules32 jules32 deleted the tutorial-00-updates-72 branch November 15, 2021 02:29
@jules32 jules32 restored the tutorial-00-updates-72 branch November 15, 2021 02:31
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