-
Notifications
You must be signed in to change notification settings - Fork 180
Update mistral.md #56
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
Conversation
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.
❌ Changes requested. Reviewed everything up to 8cc8af2 in 2 minutes and 26 seconds
More details
- Looked at
61lines of code in1files - Skipped
0files when reviewing. - Skipped posting
8drafted comments based on config settings.
1. docs/providers/mistral.md:22
- Draft comment:
Typo in table header: 'Imagage' should be 'Image'. - Reason this comment was not posted:
Marked as duplicate.
2. docs/providers/mistral.md:26
- Draft comment:
Possible typo in model name: 'ministral-8b-latest' may be intended as 'mistral-8b-latest'. - Reason this comment was not posted:
Marked as duplicate.
3. docs/providers/mistral.md:47
- Draft comment:
Consider rephrasing for clarity regarding API key usage and endpoint settings. - Reason this comment was not posted:
Comment looked like it was already resolved.
4. docs/providers/mistral.md:53
- Draft comment:
Ensure consistent terminology for API keys; consider using 'La Plateforme' if that is correct instead of 'Le Platforme'. - Reason this comment was not posted:
Marked as duplicate.
5. docs/providers/mistral.md:22
- Draft comment:
Typo: 'Imagage' in the table header should likely be 'Image' (Vision / Image support). - Reason this comment was not posted:
Marked as duplicate.
6. docs/providers/mistral.md:26
- Draft comment:
Typo: 'ministral-8b-latest' appears to be misspelled. It likely should be 'mistral-8b-latest'. - Reason this comment was not posted:
Marked as duplicate.
7. docs/providers/mistral.md:27
- Draft comment:
Typo: 'ministral-3b-latest' appears to be a typo. It likely should be 'mistral-3b-latest'. - Reason this comment was not posted:
Marked as duplicate.
8. docs/providers/mistral.md:47
- Draft comment:
Inconsistency: The API Key is referred to as 'La Plateforme API Key' earlier (line 15) but as 'Le Platforme API Key' here. Consider standardizing to 'La Plateforme API Key' for consistency. - Reason this comment was not posted:
Marked as duplicate.
Workflow ID: wflow_ftNOJDfnTgKrSbSQ
Want Ellipsis to fix these issues? Tag @ellipsis-dev in a comment. You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
Co-authored-by: ellipsis-dev[bot] <65095814+ellipsis-dev[bot]@users.noreply.github.com>
mrubens
left a comment
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.
Thank you!
Important
Update
mistral.mdto clarify API key instructions, list supported models with details, and refine Codestral usage instructions.La Plateforme API KeyandCodestral API Key.This description was created by
for 8cc8af2. It will automatically update as commits are pushed.