Skip to content
This repository was archived by the owner on Apr 1, 2025. It is now read-only.

Conversation

@KatherineCox
Copy link
Contributor

@KatherineCox KatherineCox commented Feb 5, 2025

I recently added this to AnVIL_Template and thought it could be useful for OTTR_Template as well.

Older versions of templates-to-edit.md remind people to add authors to the index.Rmd page and About.Rmd. The current one does not mention adding authors anywhere (or at least I didn't see it anywhere).

I think it makes sense for this to be a separate issue, as I'll usually add authors towards the end, when the book is at least close to done and I have a good sense of who contributed to it, rather than at the beginning, when I'm taking care of the other stuff in templates-to-edit.md

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

No spelling errors! 🎉
Comment updated at 2025-02-05-15:56:15 with changes from 5ed33cf

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

No broken url errors! 🎉
Comment updated at 2025-02-05-15:56:14 with changes from 5ed33cf

@github-actions
Copy link
Contributor

github-actions bot commented Feb 5, 2025

Re-rendered previews from the latest commit:

* note not all html features will be properly displayed in the "quick preview" but it will give you a rough idea.

Updated at 2025-02-05 with changes from the latest commit 5ed33cf

@KatherineCox KatherineCox requested a review from cansavvy February 5, 2025 15:55
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.

This is a great idea. Thanks @KatherineCox !

@cansavvy cansavvy merged commit 05804f9 into main Mar 4, 2025
11 checks passed
@cansavvy cansavvy deleted the add-authors-auto-issue branch March 4, 2025 19:25
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants