Skip to content

Conversation

@pamelafox
Copy link
Collaborator

Purpose

The azure-functions package was accidentally removed from requirements.in in PR #2835. This package is required because the Azure Function apps (document_extractor, figure_processor, text_processor) reuse the backend's requirements.txt via the copy_prepdocslib.py script, and they need the azure-functions package for the Azure Functions runtime (import azure.functions as func).

Does this introduce a breaking change?

[ ] Yes
[x] No

Does this require changes to learn.microsoft.com docs?

[ ] Yes
[x] No

Type of change

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] Documentation content changes
[ ] Other... Please describe:

Code quality checklist

See CONTRIBUTING.md for more details.

  • The current tests all pass (python -m pytest).
  • I added tests that prove my fix is effective or that my feature works
  • I ran python -m pytest --cov to verify 100% coverage of added lines
  • I ran python -m mypy to check for type errors
  • I either used the pre-commit hooks or ran ruff and black manually on my code.

The azure-functions package was accidentally removed from requirements.in
in PR Azure-Samples#2835. This package is needed because the function apps (document_extractor,
figure_processor, text_processor) use the same requirements.txt via the
copy_prepdocslib.py script, and they require azure-functions for the
Azure Functions runtime.
Copy link
Collaborator

@mattgotteiner mattgotteiner left a comment

Choose a reason for hiding this comment

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

my bad

@pamelafox pamelafox merged commit 29a6d45 into Azure-Samples:main Jan 9, 2026
40 of 41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants