-
Notifications
You must be signed in to change notification settings - Fork 1
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
Clarify cleanup steps #21
Conversation
Thanks @bethac07 Do you speak for Erin too (on this one) or should we wait for her? |
I have a philosophical question - is this template intended just for our use or are we trying to encourage broader adoption? If broader, then I would leave in metadata tracking, updating language to clarify that all users should track metadata, we have a system (w/link), and if you don't have a system, use this template provided. |
Great point – +1 to encourage broader adoption How is this text:
|
I like that sentence :) |
@ErinWeisbart all set |
Except for the metadata log is in the Setup section and your first instructions are to delete Setup. So I suggest moving it to its own section, or clarifying that somehow. Otherwise LGTM. |
Ah good catch. All set now. Merging Thank you @bethac07 @ErinWeisbart! |
I used https://github.com/broadinstitute/2021_09_01_VarChAMP-data as a reference when making these updates.