Skip to content

Conversation

@InAnYan
Copy link
Member

@InAnYan InAnYan commented Nov 26, 2025

Fixes https://github.com/JabRef/jabref-issue-melting-pot/issues/1089

Hugging Face client was marked for removal, I removed it.

Now, to use Hugging Face LLM API you just need a different base URL. And it now uses OpenAI API. Old URL doesn't work anymore.

Steps to test

  1. Select Hugging Face as provider
  2. Chat

Mandatory checks

  • I own the copyright of the code submitted and I license it under the MIT license
  • I manually tested my changes in running JabRef (always required)
  • [/] I added JUnit tests for changes (if applicable)
  • [/] I added screenshots in the PR description (if change is visible to the user)
  • [/] I described the change in CHANGELOG.md in a way that is understandable for the average user (if change is visible to the user)
  • I checked the user documentation: Is the information available and up to date? If not, I created an issue at https://github.com/JabRef/user-documentation/issues or, even better, I submitted a pull request updating file(s) in https://github.com/JabRef/user-documentation/tree/main/en.

@InAnYan InAnYan added dev: code-quality Issues related to code or architecture decisions component: ai labels Nov 26, 2025
@InAnYan
Copy link
Member Author

InAnYan commented Nov 26, 2025

I'm not sure, do I need to add a CHANGELOG.md entry?

@calixtus
Copy link
Member

No changelog entry needed. Only for user facing changes.

@calixtus calixtus enabled auto-merge November 26, 2025 22:31
@Siedlerchr Siedlerchr disabled auto-merge November 26, 2025 22:38
@Siedlerchr Siedlerchr enabled auto-merge November 26, 2025 22:38
@Siedlerchr Siedlerchr disabled auto-merge November 26, 2025 22:38
@Siedlerchr Siedlerchr enabled auto-merge November 26, 2025 22:39
@Siedlerchr Siedlerchr closed this Nov 26, 2025
auto-merge was automatically disabled November 26, 2025 22:41

Pull request was closed

@Siedlerchr Siedlerchr reopened this Nov 26, 2025
@Siedlerchr Siedlerchr enabled auto-merge November 26, 2025 22:41
@Siedlerchr
Copy link
Member

PR was stuck somehow

@Siedlerchr Siedlerchr added this pull request to the merge queue Nov 26, 2025
Merged via the queue into JabRef:main with commit c206fd4 Nov 26, 2025
135 of 146 checks passed
@Siedlerchr Siedlerchr deleted the fix/hugging-face branch November 26, 2025 23:30
shubhamk0205 pushed a commit to shubhamk0205/jabref that referenced this pull request Dec 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

component: ai dev: code-quality Issues related to code or architecture decisions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants