Want to contribute? Great. Please review the following guidelines carefully and search for existing issues before opening a new one.
Table of Contents:
- Reporting bugs
- Requesting new features
- Requesting new documentations
- Contributing code and features
- Contributing new documentations
- Updating existing documentations
- Coding conventions
- Questions?
- Update to the most recent main release; the bug may already be fixed.
- Search for existing issues; it's possible someone has already encountered this bug.
- Try to isolate the problem and include steps to reproduce it.
- Share as much information as possible (e.g. browser/OS environment, log output, stack trace, screenshots, etc.).
- Search for similar feature requests; someone may have already requested it.
- Make sure your feature fits DevDocs's vision.
- Provide a clear and detailed explanation of the feature and why it's important to add it.
Please don't open issues to request new documentations. Use the Trello board where everyone can vote.
- Search for existing issues; someone may already be working on a similar feature.
- Before embarking on any significant pull request, please open an issue describing the changes you intend to make. Otherwise you risk spending a lot of time working on something we may not want to merge. This also tells other contributors that you're working on the feature.
- Follow the coding conventions.
- If you're modifying the Ruby code, include tests and ensure they pass.
- Try to keep your pull request small and simple.
- When it makes sense, squash your commits into a single commit.
- Describe all your changes in the commit message and/or pull request.
See the docs
folder to learn how to add new documentations.
Important: the documentation's license must permit alteration, redistribution and commercial use, and the documented software must be released under an open source license. Feel free to get in touch if you're not sure if a documentation meets those requirements.
In addition to the guidelines for contributing code, the following guidelines apply to pull requests that add a new documentation:
- Your documentation must come with an official icon, in both 1x and 2x resolutions (16x16 and 32x32 pixels). This is important because icons are the only thing differentiating search results in the UI.
- DevDocs favors quality over quantity. Your documentation should only include documents that most developers may want to read semi-regularly. By reducing the number of entries, we make it easier to find other, more relevant entries.
- Remove as much content and HTML markup as possible, particularly content not associated with any entry (e.g. introduction, changelog, etc.).
- Names must be as short as possible and unique across the documentation.
- The number of types (categories) should ideally be less than 100.
If the latest documentation versions report wrongly shows a documentation to be up-to-date, please open an issue or a PR to fix it.
Important: PR's that update documentation versions that do not contain the checklist shown to you in section B of the PR template may be closed without review.
Follow the following steps to update documentations to their latest version:
- Make version/release changes in the scraper file.
- Check if the license is still correct. If you update
options[:attribution]
, also update the documentation's entry in the array inassets/javascripts/templates/pages/about_tmpl.coffee
to match. - If the documentation has a custom icon, ensure the icons in
public/icons/your_scraper_name/
are up-to-date. If you pull the updated icon from a place different than the one specified in theSOURCE
file, make sure to replace the old link with the new one. - If
self.links
is defined, check if the urls are still correct. - If the scraper inherits from
FileScraper
rather thanURLScraper
, follow the instructions for that scraper infile-scrapers.md
to obtain the source material for the scraper. - Generate the docs using
thor docs:generate <doc@version>
. - Make sure
thor docs:generate
doesn't show errors and that the documentation still works well. Verify locally that everything works and that the categorization of entries is still good. Often, updates will require code changes in the scraper or its filters to tweak some new markup in the source website or to categorize new entries. - Repeat steps 5 and 6 for all versions that you updated.
- Create a PR and make sure to fill the checklist in section B of the PR template (remove the other sections).
- two spaces; no tabs
- no trailing whitespace; blank lines should have no spaces; new line at end-of-file
- use the same coding style as the rest of the codebase
These conventions are formalized in our .editorconfig
file.
Check out EditorConfig.org to learn how to make your tools adhere to it.
If you have any questions, please feel free to ask them on the contributor chat room on Discord.