-
-
Notifications
You must be signed in to change notification settings - Fork 633
docs: Reorg and improve user experience #4629
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
Conversation
Reorganize and improve documentation to provide a better getting started and documentation navigation user experience. Included changes: - Adoption of the Diátaxis framework which prioritizes user needs and organizes documentation accordingly. This approach ensures users can easily find the information they need while providing authors with clear guidance on improving documentation for a better user experience. - Implement consistent file and folder naming optimized for SEO. Closes: aboutcode-org/www.aboutcode.org/issues/39. Signed-off-by: Thomas Steenbergen <opensource@steenbe.nl>
|
Ping @DennisClark - this pull request contains the "template" for how we want apply to other AboutCode projects. Things I at a minimum would like to see copied over to other AboutCode projects
|
|
@tsteenbe What is the plan for merging these changes. Are you waiting for @DennisClark to review the changes? |
|
@mjherzog I already pinged @AyanSinhaMahapatra over docs Slack to ask him to do the review of this PR. Yes having it pushed on SCTK RTD would be nice but @DennisClark can also build the docs locally using |
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.
Pull request overview
This PR reorganizes and improves the ScanCode documentation by adopting the Diátaxis framework and implementing consistent file and folder naming conventions optimized for SEO. The changes enhance the user experience by making documentation easier to navigate and find, while providing clearer guidance for documentation authors.
- Renamed files and references from snake_case to kebab-case (e.g.,
how_to_run_a_scan.rst→running-a-scan.rst) - Updated all internal references to use new naming conventions
- Improved heading capitalization and formatting consistency
- Enhanced code block syntax using proper directives
- Reorganized documentation structure following Diátaxis principles
Reviewed changes
Copilot reviewed 82 out of 132 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| docs/source/tutorials/visualizing-scan-results.rst | Updated reference label and heading capitalization |
| docs/source/tutorials/running-a-scan.rst | Added reference label, improved headings, updated code blocks and image references |
| docs/source/tutorials/index.rst | Simplified heading and updated file references to kebab-case |
| docs/source/tutorials/configuring-scan-output-formats.rst | Updated references and improved heading consistency |
| docs/source/tutorials/configuring-scan-detection.rst | Updated references and converted examples to code-block directives |
| docs/source/tutorials/adding-a-post-scan-plugin.rst | Updated reference labels and heading capitalization |
| docs/source/rst-snippets/* | Created new organized snippet files with improved naming and formatting |
| docs/source/reference/scancode-cli/* | Created new CLI reference structure with improved organization |
| docs/source/reference/scancode-plugins/* | Reorganized plugin documentation with consistent naming |
| docs/source/reference/scancode-reindex-licenses-cli.rst | Restructured with better headings and code examples |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Also update documentation formatting and pages layout/sections. Signed-off-by: Ayan Sinha Mahapatra <asmahapatra@aboutcode.org>
AyanSinhaMahapatra
left a comment
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 @tsteenbe. I added some changes to update the docs too
| cli-post-scan-options | ||
|
|
||
| Below is a quick reference of all ScanCode options. | ||
| For detailed descriptions and examples, please refer to the chapters in the table of contents. |
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.
We don't have any explicit links to the details page for every CLI options section, apart from this being present in the sidebar, and the toctree above is also hidden.
Instead we should include the toctree in the bottom and not keep this hidden, and also refer to the details pages for every CLI options section at the end of each quick reference sections.
| Below is a quick reference of all ScanCode options. | ||
| For detailed descriptions and examples, please refer to the chapters in the table of contents. | ||
|
|
||
| The order of the sections and all their options is the same as in the :ref:`cli-help-option`, |
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.
This is not correct, neither the order nor the actual sections are identical currently, so removing for now. The CLI has some additional grouping (like basic options are further grouped in primary scans, other scans and scan options which makes sense IMHO, if we want we can also structure the doc options similarly to avoid confusion)
| Sub-options: | ||
|
|
||
| - ``--license-clarity-score`` | ||
| - ``--tallies-key-files`` |
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.
The --classify option was moved from a pre-scan option to a post-scan option but the doc was not updated, I'm adding the update.
docs/source/reference/index.rst
Outdated
| scancode-cli/index.rst | ||
| scancode-plugins/index.rst | ||
| scancode-supported-packages | ||
| extractcode-cli |
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.
I moved extractcode and scancode-reindex-licenses into the scancode CLI section, as these are actually part of the scancode CLI IMHO. Also added the scancode-license-data CLI command here as this was missing.
| .. include:: /rst-snippets/cli-basic-options.rst | ||
| :start-line: 3 | ||
|
|
||
| ---- |
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.
Added 2 new sections which were missing and IMHO should be there:
- other CLI commands
- help/doc options

Reorganize and improve documentation to provide a better getting started and documentation navigation user experience.
Included changes:
Closes: aboutcode-org/www.aboutcode.org/issues/39.