Skip to content

Conversation

gavin-aguiar
Copy link
Contributor

Description:

Excluded azure from the packages. This fixes an issue where the azure functions version does not get updated when a version is specified in the functions requirements.txt file.
Reason for this is since azure directory has an init.py file, including azure in the package build overrides the users dependencies.

@gavin-aguiar gavin-aguiar marked this pull request as draft November 18, 2021 01:40
@github-actions
Copy link

✅ Result of Pytest Coverage

@gavin-aguiar gavin-aguiar marked this pull request as ready for review November 18, 2021 17:22
Copy link
Member

@vrdmr vrdmr left a comment

Choose a reason for hiding this comment

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

Thanks for the changes. LGTM

@gavin-aguiar gavin-aguiar merged commit ee90546 into dev Nov 18, 2021
@gavin-aguiar gavin-aguiar deleted the gaaguiar/exclude_azure_from_package branch November 18, 2021 20:29
vrdmr added a commit to Azure/azure-functions-durable-python that referenced this pull request Jun 6, 2022
Similar to Azure/azure-functions-python-library#97

Excluded azure from the packages. This fixes an issue where the `azure.function` and other azure package versions do not get updated when a version is specified in the functions requirements.txt file.

The reason for this is that the azure directory has an `__init__.py` file, including azure in the package build overrides the user's dependencies.
vrdmr added a commit to Azure/azure-functions-durable-python that referenced this pull request Jun 14, 2022
Similar to Azure/azure-functions-python-library#97

Excluded azure from the packages. This fixes an issue where the `azure.function` and other azure package versions do not get updated when a version is specified in the functions requirements.txt file.

The reason for this is that the azure directory has an `__init__.py` file, including azure in the package build overrides the user's dependencies.

Co-authored-by: gavin-aguiar <80794152+gavin-aguiar@users.noreply.github.com>
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