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

Update 06-branching.md #17

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

SultanOrazbayev
Copy link
Member

One minor point is that GitHub moved away from default master branch (now called main).

Also, once students clone the simple-website assignment, the default name for their repo will be:
5-simple-website-<NAME>.

@SultanOrazbayev SultanOrazbayev requested a review from dmil January 3, 2021 16:50
Minor changes for consistency.
@dmil
Copy link
Contributor

dmil commented Jan 4, 2021

Thanks, only thing is let's change it from 5-simple-website-<NAME> to just simple-website-<NAME> in the instructions.

@dmil
Copy link
Contributor

dmil commented Jan 4, 2021

Actually, now that I think about it, simple-website isn't a Github classroom assignment yet. So it won't clone into a simple-website-<NAME> folder, just simple-website.

@SultanOrazbayev
Copy link
Member Author

Actually, now that I think about it, simple-website isn't a Github classroom assignment yet. So it won't clone into a simple-website-<NAME> folder, just simple-website.

Hmm, no, there is an assignment for that repo: https://classroom.github.com/a/IcLXBRpw
(but it's treated as an assignment only to simplify collection of all repos, there is no autograding)

@dmil
Copy link
Contributor

dmil commented Jan 4, 2021

I see ... I think let's remove the assignment on this one. Since one of the learning objectives here is to better understand GitHub, let's have them fork it directly. That also simplifies the naming structure.

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