Skip to content

Conversation

@bielsnohr
Copy link
Collaborator

This PR implements the antecedent changes in the lesson to enable changes in #226 to work properly for learners. It should only be merged after that PR because it is based on top of that source branch. Closes #249 .

Instruct learners to create a fork of the project repository instead of a copy
of a template in 11-software-project.md
@bielsnohr bielsnohr self-assigned this Oct 25, 2023
@anenadic anenadic requested a review from steve-crouch November 7, 2023 09:20
@steve-crouch steve-crouch changed the base branch from gh-pages to dev November 21, 2023 12:16
@anenadic
Copy link
Collaborator

anenadic commented Nov 21, 2023

Updated inflammation template repo with branches to use when reviewing this PR (that also needs to be synched with the original inflammation template repo): https://github.com/thomaskileyukaea/python-intermediate-inflammation

@bielsnohr
Copy link
Collaborator Author

Updated inflammation template repo with branches to use when reviewing this PR (that also needs to be synched with the original inflammation template repo): https://github.com/thomaskileyukaea/python-intermediate-inflammation

@anenadic we agreed in the maintainers sprint that it is probably actually best to create a complete new and distinct learner project repo to avoid problems with any forked versions of this course. @steve-crouch please correct me on that if I misinterpreted.

Once we create that new repository, we can update the links here to point to it.

@anenadic anenadic mentioned this pull request Nov 24, 2023
@anenadic anenadic changed the base branch from dev to 249-fork-course-project November 27, 2023 11:53
@anenadic anenadic requested review from anenadic and removed request for steve-crouch November 27, 2023 11:55
@anenadic anenadic changed the base branch from 249-fork-course-project to dev November 27, 2023 12:00
@anenadic anenadic changed the base branch from dev to 249-fork-course-project November 27, 2023 12:02
Copy link
Collaborator

@anenadic anenadic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merging this to a new branch on the base repo. The next is merging this to dev after a round of reviews and then to gh-pages.

@anenadic anenadic merged commit 2963d50 into 249-fork-course-project Nov 27, 2023
@anenadic anenadic deleted the bielsnohr/249-fork-course-project branch October 3, 2024 12:43
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.

Change initial retrieval of course project to be a fork rather than template copy

3 participants