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

when starting course, replace README #667

Merged
merged 1 commit into from
Aug 10, 2023
Merged

when starting course, replace README #667

merged 1 commit into from
Aug 10, 2023

Conversation

KatherineCox
Copy link
Contributor

Currently the README is doing double duty - describing OTTR_Template and trying to serve as a reasonable starting point for a README for a course.

I've been working on a C-MOOR branded copy of OTTR_Template, and thought it would be useful to automatically replace the README when starting a course, so that those two concerns can be split up.

This PR

  • adds a course_README_template.md to hold the starting point for a course's README
  • updates starting-course.yml to replace README.md with course_README_template.md
  • moves the course-related content from the existing README.md into course_README_template.md

@cansavvy If you want to make changes to this PR, go for it - I'm handing it off to you. I thought you might have other changes you would want to make to the README content, now that's being split up. I just took the existing content and distributed it between the two files (and updated the link to point to ottrproject.org).

@KatherineCox KatherineCox requested a review from cansavvy August 9, 2023 19:43
@github-actions
Copy link
Contributor

github-actions bot commented Aug 9, 2023

No broken urls! 🎉
Comment updated at 2023-08-09 with changes from 3e6bf6a

Copy link
Collaborator

@cansavvy cansavvy left a comment

Choose a reason for hiding this comment

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

@KatherineCox This is excellent! Thanks for doing this! Great idea.

@cansavvy cansavvy merged commit 9a8169e into main Aug 10, 2023
@cansavvy cansavvy deleted the KCox/swap-readme branch August 10, 2023 15:08
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