-
Notifications
You must be signed in to change notification settings - Fork 51
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
update contribution guidelines for docs #467
Comments
Many thanks for your suggestions. Good points. It makes sense to make clear on each repository what it is for, and add that to our contribution page and/or the docs.
As for the linter, when cloning the repo within GitHub, one just needs to enable GitHub actions for the own repo (I think that is not the case by default). Since the actions config is part of the repo, it should then run automatically on each commit: https://github.com/MichaIng/DietPi-Docs/blob/master/.github/workflows/actions.yml That is not true for CodeFactor checks, which is an app that requires an account. But we might convert that into GitHub actions as well, to have everything in one place. |
Regarding the linter (Markdown checker)What should I do before issueing a PR? Then, select the "..." and "View workflow file" which belongs to your push operation: Then, click on "Actions" and examine the output of the checks: Remarks:
|
Regarding editing MarkdownHere we may collect some useful hints for editing the Markdown files. AdmonitionsIf you want to use admonitions: See mkdocs reference. Keyboard keysIf you want to use keyboard keys, see PyMdown Extension docu and mkdocs reference. |
little weirdies
I actually started this a bit already. (Those who can't do, teach.)
**
Am not sure exactly where it might live. On repos' readmes there are https://github.com/MichaIng/DietPi#contributing and https://github.com/MichaIng/DietPi-Docs#contributing A separate file might be better? Some of it might be useful to the rest of the project but mostly it would be too basic.
please add
I will add as I come across things.
The text was updated successfully, but these errors were encountered: