-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Extend contributors guide #4485
Comments
Now this issue is regarding the contributors, but what I had asked in #4288 (comment) is more user-facing, about feature caveats and helpful info like #5501. They are scattered around thousands of issues, and we frequently need to dig around to provide these for users. Wouldn't it be optimal if we can enable wiki in this repo so we have a central place to store these knowledge instead of opening obscure aggregation issues like #2960? |
Good idea, let’s move that content to the wiki. Even the contributing guide can be moved to the wiki to facilitate contributions. Keep in mind however that the vast majority of the knowledge will continue to be lost in the conversations and commits, because nobody will spend time to move it out after a PR has been merged. It would be interesting to create Probably not 😂 it’s best to just move those 3/4 issues into the wiki. As for the CSS stuff, I would not spend time on that at all. If enough people complain, we just make the feature disableable, it makes no sense to maintain undo styles. Our intention is to just avoid having to move every single CSS change to a JavaScript file. |
I will spend time maintaining the wiki 😉 I've complained lack of detailed docs a few times already.
Writing reverts still takes time and effort, however simple it is. Put them somewhere easy to find save both users and our time. |
We can also move this issue to the wiki: And add more links to core test URLs like |
Following #4200
The project could include further documentation: https://github.com/sindresorhus/refined-github/blob/main/collaborators.md
The text was updated successfully, but these errors were encountered: