Skip to content

[608] Migrate Documentation to GitHub Pages#641

Merged
hk21702 merged 16 commits intoRimSort:mainfrom
hk21702:jekyll-docs-migration
Nov 14, 2024
Merged

[608] Migrate Documentation to GitHub Pages#641
hk21702 merged 16 commits intoRimSort:mainfrom
hk21702:jekyll-docs-migration

Conversation

@hk21702
Copy link
Copy Markdown
Collaborator

@hk21702 hk21702 commented Oct 14, 2024

Partially implements #608. Migrates documentation to use GitHub pages using the "Just the Docs" Jekyll theme. Also, includes a better formatted README file.

All information from the old wiki has been migrated. Assets have not been updated. Some content has been improved and or added, but most is the same from the old wiki.

Some pages were broken up into sub-pages to take better advantage of the flexibility that GitHub pages gives us.
Everything other than the 404 page is written in markdown.

Wiki link within RimSort has not been updated in case there is an issue with first deployment.

image
image

@hk21702 hk21702 added the documentation 📚 related to Docs like wiki page README, changelog label Oct 14, 2024
@hk21702 hk21702 self-assigned this Oct 14, 2024
@github-actions github-actions bot added the DevOps 💻 Related to development automation, auto building, auto actions, and other CI/CD things label Oct 14, 2024
@hk21702 hk21702 force-pushed the jekyll-docs-migration branch 2 times, most recently from c55554e to cca9950 Compare October 14, 2024 01:57
@hk21702 hk21702 force-pushed the jekyll-docs-migration branch 2 times, most recently from 4924658 to 753d9ea Compare October 14, 2024 02:20
@hk21702 hk21702 force-pushed the jekyll-docs-migration branch from 753d9ea to 928a70a Compare October 14, 2024 14:44
Copy link
Copy Markdown
Collaborator

@Patryk-Malinowski Patryk-Malinowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What do you think about removing the table of contents for User and Dev Guide? And I only mean the table in the in the index.md files. I don't know how full these will get but I still imagine it will mainly be linking to other pages and would look cleaner and less cluttered without the TOC?

image

image

I feel like this gets worse the more links are added and can imagine a new user opening the page and being confused/overwhelmed? This can also maybe give the impression of the TOC links being different from the above links due to some differences in naming? Like Using the Rule Editor and Rule Editor could make a user think they are different links at first?

image

#########################################

Would it be a good idea to move the External Metadata away from basic usage?
In the image below, maybe at the end of that part say something like 'to learn more about Community rules...' etc. with a link so the users can read up about the steam workshop and community rules databases?
image

However at the same time I guess we want to highlight why they should use them... so maybe a way of saying 'we strongly recommend you look at this link and download/enable this'.

Or maybe keep a short and really simple paragraph on each in Basic Usage explaining what they are used for and that we strongly recommend users set it up with a link leading to the steps on how to do this?

And we can keep more detailed information away from Basic Usage if they want to give that a read?

@Patryk-Malinowski
Copy link
Copy Markdown
Collaborator

Oh and otherwise I think it looks great. Looking forward to when this gets merged :)

@hk21702
Copy link
Copy Markdown
Collaborator Author

hk21702 commented Nov 13, 2024

What do you think about removing the table of contents for User and Dev Guide? And I only mean the table in the in the index.md files. I don't know how full these will get but I still imagine it will mainly be linking to other pages and would look cleaner and less cluttered without the TOC?

I initially wanted to do this, but as far as I can tell, you have to have a parent page when you categorize stuff like this. I could be wrong though and I'll double check.

@hk21702
Copy link
Copy Markdown
Collaborator Author

hk21702 commented Nov 13, 2024

Would it be a good idea to move the External Metadata away from basic usage?
In the image below, maybe at the end of that part say something like 'to learn more about Community rules...' etc. with a link so the users can read up about the steam workshop and community rules databases?

We can. I'll look at changing it in this pr, but the goal was to just migrate existing stuff from the old wiki.

@Patryk-Malinowski
Copy link
Copy Markdown
Collaborator

I initially wanted to do this, but as far as I can tell, you have to have a parent page when you categorize stuff like this. I could be wrong though and I'll double check.

Just to make sure, I meant keep the parent/index.md files but remove TOC. I think I just used has_toc: false at the top of index.md to remove the TOC.

@hk21702
Copy link
Copy Markdown
Collaborator Author

hk21702 commented Nov 13, 2024

Just to make sure, I meant keep the parent/index.md files but remove TOC. I think I just used has_toc: false at the top of index.md to remove the TOC.

Oh sorry, yeah can do that. Weird... for some reason that wasn't rendering at all for me on my side.

@hk21702
Copy link
Copy Markdown
Collaborator Author

hk21702 commented Nov 13, 2024

Would it be a good idea to move the External Metadata away from basic usage?
In the image below, maybe at the end of that part say something like 'to learn more about Community rules...' etc. with a link so the users can read up about the steam workshop and community rules databases?

We can. I'll look at changing it in this pr, but the goal was to just migrate existing stuff from the old wiki.

Looking at this again, I will just simplify it a little. Moving it out entirely is a little overkill as we already have a databases page. But, with the new format, it can be organized a lot better to make it easier on the eyes.

@hk21702
Copy link
Copy Markdown
Collaborator Author

hk21702 commented Nov 13, 2024

Just to make sure, I meant keep the parent/index.md files but remove TOC. I think I just used has_toc: false at the top of index.md to remove the TOC.

Oh sorry, yeah can do that. Weird... for some reason that wasn't rendering at all for me on my side.

Honestly, the automatic rendering you see here should be preferred, as it is automatic. So, instead, I'm going to axe the manual stuff I made back when I couldn't get it to show up on its own.

@hk21702 hk21702 force-pushed the jekyll-docs-migration branch from 0c7958a to 42be893 Compare November 13, 2024 07:05
@hk21702
Copy link
Copy Markdown
Collaborator Author

hk21702 commented Nov 13, 2024

I updated a bit more content, but for this PR I'm going to pause on any content updates unless it is critical.

@hk21702 hk21702 linked an issue Nov 13, 2024 that may be closed by this pull request
11 tasks
Copy link
Copy Markdown
Collaborator

@Patryk-Malinowski Patryk-Malinowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removing the manual links and just leaving the generated TOC was definitely the right call. Will save us the hassle of remembering and adding new links etc. as more subpages are added.

The external metadata part looks great too, I definitely think it is easier to digest now.

@Patryk-Malinowski
Copy link
Copy Markdown
Collaborator

We can. I'll look at changing it in this pr, but the goal was to just migrate existing stuff from the old wiki.

Yeah sorry that might have been going past the 'structure review'. Just wanted to mention it. Obviously once we are using this we can start updating the content and adding more etc.

@hk21702 hk21702 merged commit 831d3eb into RimSort:main Nov 14, 2024
github-merge-queue bot pushed a commit that referenced this pull request Mar 23, 2026
Bumps [pytest-cov](https://github.com/pytest-dev/pytest-cov) from 7.0.0
to 7.1.0.
<details>
<summary>Changelog</summary>
<p><em>Sourced from <a
href="https://github.com/pytest-dev/pytest-cov/blob/master/CHANGELOG.rst">pytest-cov's
changelog</a>.</em></p>
<blockquote>
<h2>7.1.0 (2026-03-21)</h2>
<ul>
<li>
<p>Fixed total coverage computation to always be consistent, regardless
of reporting settings.
Previously some reports could produce different total counts, and
consequently can make --cov-fail-under behave different depending on
reporting options.
See <code>[#641](pytest-dev/pytest-cov#641)
&lt;https://github.com/pytest-dev/pytest-cov/issues/641&gt;</code>_.</p>
</li>
<li>
<p>Improve handling of ResourceWarning from sqlite3.</p>
<p>The plugin adds warning filter for sqlite3
<code>ResourceWarning</code> unclosed database (since 6.2.0).
It checks if there is already existing plugin for this message by
comparing filter regular expression.
When filter is specified on command line the message is escaped and does
not match an expected message.
A check for an escaped regular expression is added to handle this
case.</p>
<p>With this fix one can suppress <code>ResourceWarning</code> from
sqlite3 from command line::</p>
<p>pytest -W &quot;ignore:unclosed database in &lt;sqlite3.Connection
object at:ResourceWarning&quot; ...</p>
</li>
<li>
<p>Various improvements to documentation.
Contributed by Art Pelling in
<code>[#718](pytest-dev/pytest-cov#718)
&lt;https://github.com/pytest-dev/pytest-cov/pull/718&gt;</code>_ and
&quot;vivodi&quot; in
<code>[#738](pytest-dev/pytest-cov#738)
&lt;https://github.com/pytest-dev/pytest-cov/pull/738&gt;</code><em>.
Also closed
<code>[#736](pytest-dev/pytest-cov#736)
&lt;https://github.com/pytest-dev/pytest-cov/issues/736&gt;</code></em>.</p>
</li>
<li>
<p>Fixed some assertions in tests.
Contributed by in Markéta Machová in
<code>[#722](pytest-dev/pytest-cov#722)
&lt;https://github.com/pytest-dev/pytest-cov/pull/722&gt;</code>_.</p>
</li>
<li>
<p>Removed unnecessary coverage configuration copying (meant as a backup
because reporting commands had configuration side-effects before
coverage 5.0).</p>
</li>
</ul>
</blockquote>
</details>
<details>
<summary>Commits</summary>
<ul>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/66c8a526b1246b5eb8fb1bc218878131bc628622"><code>66c8a52</code></a>
Bump version: 7.0.0 → 7.1.0</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/f7076624784332594aa4cb3585d4757d295db15e"><code>f707662</code></a>
Make the examples use pypy 3.11.</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/6049a7847872e3139e6c82e93787123df5dc8672"><code>6049a78</code></a>
Make context test use the old ctracer (seems the new sysmon tracer
behaves di...</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/8ebf20bbbc73478b3f8fd36d30237d9ea083f06b"><code>8ebf20b</code></a>
Update changelog.</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/861d30e60d571f97259c6b718b71c819d5dbc3b9"><code>861d30e</code></a>
Remove the backup context manager - shouldn't be needed since coverage
5.0, ...</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/fd4c956014035527f0c3c8d7faef3f8cfdadac7f"><code>fd4c956</code></a>
Pass the precision on the nulled total (seems that there's some caching
goion...</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/78c9c4ecb005faf4962fd86ff7bf9c9cce9554d6"><code>78c9c4e</code></a>
Only run the 3.9 on older deps.</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/4849a922e8be725c662a3d9175da571ace6545dc"><code>4849a92</code></a>
Punctuation.</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/197c35e2f37031fd1927715307ab6eed7cb3d2b7"><code>197c35e</code></a>
Update changelog and hopefully I don't forget to publish release again
:))</li>
<li><a
href="https://github.com/pytest-dev/pytest-cov/commit/14dc1c92d44108384e39803888635fdbfc578b7f"><code>14dc1c9</code></a>
Update examples to use 3.11 and make the adhoc layout example look a bit
more...</li>
<li>Additional commits viewable in <a
href="https://github.com/pytest-dev/pytest-cov/compare/v7.0.0...v7.1.0">compare
view</a></li>
</ul>
</details>
<br />


[![Dependabot compatibility
score](https://dependabot-badges.githubapp.com/badges/compatibility_score?dependency-name=pytest-cov&package-manager=uv&previous-version=7.0.0&new-version=7.1.0)](https://docs.github.com/en/github/managing-security-vulnerabilities/about-dependabot-security-updates#about-compatibility-scores)

Dependabot will resolve any conflicts with this PR as long as you don't
alter it yourself. You can also trigger a rebase manually by commenting
`@dependabot rebase`.

[//]: # (dependabot-automerge-start)
[//]: # (dependabot-automerge-end)

---

<details>
<summary>Dependabot commands and options</summary>
<br />

You can trigger Dependabot actions by commenting on this PR:
- `@dependabot rebase` will rebase this PR
- `@dependabot recreate` will recreate this PR, overwriting any edits
that have been made to it
- `@dependabot show <dependency name> ignore conditions` will show all
of the ignore conditions of the specified dependency
- `@dependabot ignore this major version` will close this PR and stop
Dependabot creating any more for this major version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this minor version` will close this PR and stop
Dependabot creating any more for this minor version (unless you reopen
the PR or upgrade to it yourself)
- `@dependabot ignore this dependency` will close this PR and stop
Dependabot creating any more for this dependency (unless you reopen the
PR or upgrade to it yourself)


</details>

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DevOps 💻 Related to development automation, auto building, auto actions, and other CI/CD things documentation 📚 related to Docs like wiki page README, changelog

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Suggestion: Update Wiki

2 participants