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

Can simplify cloning repo & making personal repo remote with a fork instead #3

Open
Tr3ntroxs opened this issue Feb 3, 2023 · 2 comments

Comments

@Tr3ntroxs
Copy link

No description provided.

@Gerzer
Copy link
Collaborator

Gerzer commented Feb 4, 2023

Unfortunately, a fork won’t work because you can’t make forks of public repositories private. However, GitHub offers an “import” feature, which does support private repositories, for transferring code from one repository to another. Perhaps you or someone else could modify the instructions to mention that as an option instead of cloning the source repository directly.

That said, being able to configure multiple remotes in Git is an important skill. Currently, the exam effectively tests that skill, but implementing this suggestion would undermine that, which might be undesirable. On the other hand, the auto-grader can’t itself check which method the student used, so adding instructions for the “import” feature would merely make explicit something that students can already do.

@wdturner
Copy link

wdturner commented Feb 7, 2023

Remotes are useful, particularly when you are working with a group like RCOS which may want to work together outside of the normal development path for the project ... I.e. perhaps the Open Horizons team at RPI would want to work out of a common fork outside of IBM and then make PRs from this common fork into the main fork of Open Horizon. I think learning remotes are useful, but the description needs to be better explained.

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

3 participants