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

Documented process to update answer store? #2

Open
munkm opened this issue Apr 28, 2020 · 2 comments
Open

Documented process to update answer store? #2

munkm opened this issue Apr 28, 2020 · 2 comments

Comments

@munkm
Copy link
Member

munkm commented Apr 28, 2020

I think we should add how to contribute answers to this answer store repo somewhere. In our main docs for updating answers, but also maybe a step-by-step guide in the readme of this repo? What do you think?

Some other thoughts:

  • reviewing PRs in this repo should require only one approving review (ideally from a reviewer on the main PR).
  • each PR to this repo should include a link to the supporting PR in the main yt repo.
  • I also propose that we require contributors to link to the failed diff for the updated answers generated by travis in their PR in this repo. That will make reviewing these answers easier to compare without doing any detective work.
  • we can make a PR template for this repo that includes a checklist for the required link to yt.upload and a link to the PR.
@neutrinoceros
Copy link
Member

To answer your first question, I think linking the website-guide in the local readme would suffice !

@neutrinoceros
Copy link
Member

I once wrote an detailed note to explain the process as a GitHub comment, linking it here for posterity
yt-project/yt#3282 (comment)

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

2 participants