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

GPT4All: Add version v3.3.0 #14084

Open
wants to merge 7 commits into
base: master
Choose a base branch
from

Conversation

guillermodotn
Copy link

@guillermodotn guillermodotn commented Sep 25, 2024

Adding manifest for installation of GPT4All.

Closes #11531

Copy link
Contributor

Your changes do not pass checks.

gpt4all

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@guillermodotn
Copy link
Author

/verify manifest checkver

Copy link
Contributor

All changes look good.

Wait for review from human collaborators.

gpt4all

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@aliesbelik
Copy link
Collaborator

  • Guess bin/shortcuts fields could be added as well.
  • Please update to the latest version.

@guillermodotn
Copy link
Author

The installer automatically creates the shortcut, so including the shortcut property would create two of them, which I find unnecessary. I have added the following section to the manifest explaining why the shortcut property isn't included:

"##": [
        [
            "Shortcut section omitted as it is automatically created by the installer.",
            "bin/bin/chat.exe",
            "GPT4All"
        ]
    ],

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Request]: Add GPT4All
2 participants