-
Notifications
You must be signed in to change notification settings - Fork 303
Copy prompt from a remote folder in docs#2715
Copy prompt from a remote folder in docs #2715atharvadeosthale wants to merge 5 commits intoai-docs from
Conversation
Important
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.
You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.
Use the checkbox below for a quick retry:
- 🔍 Trigger review
✨ Finishing touches
🧪 Generate unit tests (beta)
- Create PR with unit tests
- Post copyable unit tests in a comment
- Commit unit tests in branch
prompt-markdoc
Tip
Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.
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.
Comment @coderabbitai help to get the list of available commands and usage tips.
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.
do we not have a frontmatter parser somewhere already? 👀
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 would like to avoid adding a preprocessor. Only if it would affect 90% of the markdown files it would be suitable.
cant we just pass the routeId as frontmatter to
website/src/lib/utils/routePrompts.ts
Line 27 in 9c8868b
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 tried having an implementation where it just uses a markdown (prompt.md) file at a route. The problem with that is that we lose all styling because it uses some other markdown processor (I think it-markdown). I checked how partials are processed, and it's through another pre-processor. Since I couldn't go around any other way, I decided to do this. Lmk ur thoughts @TorstenDittmann
No description provided.