-
Notifications
You must be signed in to change notification settings - Fork 10
docs: add prompt-writing guidance for AI assistants #662
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
base: main
Are you sure you want to change the base?
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -4,7 +4,7 @@ | |||||
| <tr> | ||||||
| <td><b>6 minutes read</b></td> | ||||||
| <td style={{ paddingLeft: 40 }}><b> Level: Advanced</b></td> | ||||||
| <td style={{ paddingLeft: 40 }}><b>Last Updated: April 2026</b></td> | ||||||
| <td style={{ paddingLeft: 40 }}><b>Last Updated: July 2026</b></td> | ||||||
| </tr> | ||||||
| </table> | ||||||
| </h3> | ||||||
|
|
@@ -47,7 +47,7 @@ Define the following parameters: | |||||
|
|
||||||
|
|
||||||
| - Provide a system prompt in the `Instructions` field. | ||||||
| *[Click Here](https://glific.org/a-simple-guide-to-using-large-language-models/#prompt) to read more on prompt engineering.* | ||||||
| *[Click Here](https://glific.org/a-simple-guide-to-using-large-language-models/#prompt) to read more on prompt engineering, or see [Writing effective instructions](https://glific.github.io/docs/docs/Integrations/Creating%20and%20modifying%20assistants%20in%20Glific/#writing-effective-instructions) for a prompt template and examples.* | ||||||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win Replace “Click Here” with descriptive link text. Use text such as “Read a simple guide to prompt engineering” so the link’s destination is clear without relying on surrounding context. Proposed fix-*[Click Here](https://glific.org/a-simple-guide-to-using-large-language-models/#prompt)
+*[Read a simple guide to prompt engineering](https://glific.org/a-simple-guide-to-using-large-language-models/#prompt)📝 Committable suggestion
Suggested change
🧰 Tools🪛 markdownlint-cli2 (0.23.0)[warning] 50-50: Link text should be descriptive (MD059, descriptive-link-text) 🤖 Prompt for AI AgentsSource: Linters/SAST tools |
||||||
| - Files (.csv, .doc, .docx, .html, .java, .md, .pdf, .pptx, .txt) can be uploaded by clicking on `Manage Files`. | ||||||
| These files will be utilized by the assistant to generate responses and each file will take approx 15 secs to upload. | ||||||
|
|
||||||
|
|
||||||
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.
📐 Maintainability & Code Quality | 🟡 Minor | ⚡ Quick win
Specify a language for both fenced prompt examples.
Add
text(ormarkdown) after each opening fence so the documentation passes MD040 and renders consistently.Proposed fix
Also applies to: 57-65
🧰 Tools
🪛 markdownlint-cli2 (0.23.0)
[warning] 45-45: Fenced code blocks should have a language specified
(MD040, fenced-code-language)
🤖 Prompt for AI Agents
Source: Linters/SAST tools