-
Notifications
You must be signed in to change notification settings - Fork 1
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
add include component #119
Conversation
WalkthroughThis pull request introduces a new mechanism for including external Markdown content within a document. A new Vue component Changes
Sequence DiagramsequenceDiagram
participant User
participant Include.vue
participant ContentRenderer
participant queryContent
User->>Include.vue: Provide path prop
Include.vue->>queryContent: Request content for path
queryContent-->>Include.vue: Return content data
Include.vue->>ContentRenderer: Render fetched content
Possibly related PRs
Suggested reviewers
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
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.
Actionable comments posted: 2
🧹 Nitpick comments (1)
doc/components.md (1)
7-12
: Enhance documentation with more details and examples.The documentation could be more comprehensive. Consider adding:
- Purpose and use cases for the include component
- Limitations or restrictions (if any)
- Multiple examples showing different scenarios
- Language specification for the code block (as flagged by markdownlint)
Here's a suggested improvement:
## Инклюд -``` +Компонент для включения содержимого других Markdown файлов в текущий документ. + +### Использование + +```md :include{path="Путь до md файла внутри локали (например "/_integrators/abrom"}+### Примеры
+
+1. Включение файла из той же директории:
+md +:include{path="./installation.md"} +
+
+2. Включение файла из другой локали:
+md +:include{path="/_integrators/setup.md"} +
<details> <summary>🧰 Tools</summary> <details> <summary>🪛 Markdownlint (0.37.0)</summary> 9-9: null Fenced code blocks should have a language specified (MD040, fenced-code-language) </details> </details> </blockquote></details> </blockquote></details> <details> <summary>📜 Review details</summary> **Configuration used: CodeRabbit UI** **Review profile: CHILL** **Plan: Pro** <details> <summary>📥 Commits</summary> Reviewing files that changed from the base of the PR and between 4f23f7229e3752dd1676967cb3dc9aeb9ea05603 and 77286d1636e137bbb2e1f84aed5efd1d75999168. </details> <details> <summary>📒 Files selected for processing (2)</summary> * `components/content/Include.vue` (1 hunks) * `doc/components.md` (1 hunks) </details> <details> <summary>🧰 Additional context used</summary> <details> <summary>🪛 Markdownlint (0.37.0)</summary> <details> <summary>doc/components.md</summary> 9-9: null Fenced code blocks should have a language specified (MD040, fenced-code-language) </details> </details> </details> <details> <summary>⏰ Context from checks skipped due to timeout of 90000ms (1)</summary> * GitHub Check: Codacy Static Code Analysis </details> </details> <!-- This is an auto-generated comment by CodeRabbit for review status -->
Добавил компонент инклюд
Summary by CodeRabbit
New Features
Include.vue
for dynamically rendering content from specified pathsDocumentation