Skip to content
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

Chat readme: Updates features and removes wish-list & gpt-j #2698

Merged
merged 1 commit into from
Jul 25, 2024

Conversation

ThiloteE
Copy link
Collaborator

What changed:

Updates Readme.md of gpt4all-chat

Motivation for pull-request:

  1. Remove potential cause for duplication and conflicts between documentation on website and the wiki.
  2. Readme should be future proof, so handle wish-list and roadmap via issue tracker.

Describe your changes

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code.
  • If it is a core feature, I have added thorough tests.
  • I have added thorough documentation for my code.
  • I have tagged PR with relevant project labels. I acknowledge that a PR without labels may be dismissed.
  • If this PR addresses a bug, I have provided both a screenshot/video of the original bug and the working solution.

### What changed:

Updates Readme.md of gpt4all-chat

- updates features
- removes feature wish-list
- removes any mention of gpt-j since support for it has been removed with #2676

### Motivation for pull-request:

1. Remove potential cause for duplication and conflicts between documentation on website and the wiki.
2. Readme should be future proof, so handle wish-list and roadmap via issue tracker.

Signed-off-by: ThiloteE <73715071+ThiloteE@users.noreply.github.com>
@ThiloteE ThiloteE added the documentation Improvements or additions to documentation label Jul 19, 2024
@manyoso manyoso requested a review from mcembalest July 19, 2024 20:00
Copy link
Contributor

@mcembalest mcembalest left a comment

Choose a reason for hiding this comment

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

LGTM

@ThiloteE ThiloteE changed the title Update README.md of gpt4all-chat Chat: README.md - Update list of features - removes wish-list - remove mention of gpt-j Jul 24, 2024
@ThiloteE ThiloteE changed the title Chat: README.md - Update list of features - removes wish-list - remove mention of gpt-j Chat: README.md - Updates features - removes wish-list & gpt-j Jul 24, 2024
@ThiloteE ThiloteE changed the title Chat: README.md - Updates features - removes wish-list & gpt-j Chat readme: - Updates features - removes wish-list & gpt-j Jul 24, 2024
@ThiloteE ThiloteE changed the title Chat readme: - Updates features - removes wish-list & gpt-j Chat readme: Updates features and removes wish-list & gpt-j Jul 24, 2024
@manyoso manyoso merged commit 6b97d0f into main Jul 25, 2024
6 of 12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants