Skip to content

Conversation

aksg87
Copy link
Collaborator

@aksg87 aksg87 commented Aug 13, 2025

Fix test failure by ensuring providers are loaded for registry when using lx.extract() in tests.

The test was failing in CI because the provider registry was empty - providers need to be explicitly loaded in the test environment.

@github-actions github-actions bot added the size/XS Pull request with less than 50 lines changed label Aug 13, 2025
Copy link

No linked issues found. Please link an issue in your pull request description or title.

Per our Contributing Guidelines, all PRs must:

  • Reference an issue with one of:
    • Closing keywords: Fixes #123, Closes #123, Resolves #123 (auto-closes on merge in the same repository)
    • Reference keywords: Related to #123, Refs #123, Part of #123, See #123 (links without closing)
  • The linked issue should have 5+ 👍 reactions from unique users (excluding bots and the PR author)
  • Include discussion demonstrating the importance of the change

You can also use cross-repo references like owner/repo#123 or full URLs.

@aksg87 aksg87 force-pushed the fix/test-import-order branch from ac6c17b to 32a2ad5 Compare August 13, 2025 09:51
@github-actions github-actions bot added the size/S Pull request with 50-150 lines changed label Aug 13, 2025
@aksg87 aksg87 merged commit e056a85 into main Aug 13, 2025
12 of 14 checks passed
sinnaj pushed a commit to sinnaj/langextract that referenced this pull request Sep 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/S Pull request with 50-150 lines changed size/XS Pull request with less than 50 lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant