Skip to content

Conversation

@thephez
Copy link
Collaborator

@thephez thephez commented Jan 21, 2026

No longer provides Dash services.

Also updated/re-organized exclude patterns

Preview build: https://dash-docs--557.org.readthedocs.build/en/557/

Summary by CodeRabbit

  • Documentation

    • Removed Pool of Stake hosting provider from the master nodes hosting services list.
  • Chores

    • Updated documentation configuration to refine build process management.

✏️ Tip: You can customize this high-level summary in your review settings.

No longer provides Dash services.

Also updated/re-organized exclude patterns
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Jan 21, 2026

📝 Walkthrough

Walkthrough

Configuration update to expand Sphinx-excluded paths and file patterns, combined with removal of a hosting service provider entry from the masternodes documentation.

Changes

Cohort / File(s) Summary
Build Configuration
conf.py
Expanded exclude_patterns with new entries: .devcontainer, .DS_Store, local/*, scripts/*, transifex/*, venv/*, dash-core-src, CLAUDE.md. Reorganized and removed duplicate/rearranged prior patterns.
Documentation Content
docs/user/masternodes/hosting.rst
Removed Pool of Stake hosting service provider entry including image directive, metadata (site, email, Twitter, Telegram), and associated configuration details (19 lines removed).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The PR title clearly describes the main change: removing 'pool of stake' from the hosting list. It aligns well with the primary objective and the file changes.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)
conf.py (1)

44-48: Add .md support to source_suffix for Markdown documentation processing.

MyST parser is enabled, but source_suffix is set to .rst only, preventing .md files from being processed. Update it to support both extensions:

🔧 Proposed fix
-source_suffix = '.rst'
+source_suffix = {
+    '.rst': 'restructuredtext',
+    '.md': 'markdown',
+}

This aligns with the coding guidelines requiring both .rst and .md support via MyST parser.

@thephez thephez merged commit 0574bd9 into dashpay:23.0.0 Jan 21, 2026
2 checks passed
@thephez thephez deleted the hosting-update-2026-01-21 branch January 21, 2026 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant