-
Notifications
You must be signed in to change notification settings - Fork 31
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
Move chapter owners to page maintainers #297
Conversation
@@ -65,3 +65,6 @@ | |||
* Generic Auto Tuners: | |||
* [Kernel Tuner](https://github.com/benvanwerkhoven/kernel_tuner) (Python) | |||
* [CLTune](https://github.com/CNugteren/CLTune) (C++) | |||
|
|||
|
|||
*Page maintainer: unmaintained* |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@benvanwerkhoven Would you still be interested in maintaining the OpenCL/CUDA page now that you have a new place to work?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi Bouwe! Indeed, I think it might be better if someone else, for example @isazi, takes over.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @bouweandela I will take over that from Ben. Let me know what else you need from me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @isazi! Being a page maintainer involves checking once a year or so if the content is still up to date (and updating it if it turns out to be outdated) and if there is a pull request from someone else you will be asked to review it.
What do you think about putting the maintainers on top of the page instead to make them a bit more visible? We're going to be asking them periodically for some work, so I feel it's fair to give them some clear proper credit. |
Replace bottom by top
f443808
to
ca79bf1
Compare
Done in ca79bf1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! We should probably add maintainers to almost all pages (except the front page maybe), but that can be a separate PR.
Agreed. I added them to all pages that we plan to keep, though I may have missed some. |
Add a maintainer to every page that we plan to keep (see #286 for a list).