Skip to content

Comments

Add issue link for TODO wrt Azure integration pinned dependencies#33064

Merged
potiuk merged 1 commit intoapache:mainfrom
astronomer:add-azure-pinned-dependency-issue-link
Aug 3, 2023
Merged

Add issue link for TODO wrt Azure integration pinned dependencies#33064
potiuk merged 1 commit intoapache:mainfrom
astronomer:add-azure-pinned-dependency-issue-link

Conversation

@pankajkoti
Copy link
Member

While working on issue #33025, it was unclear on why we've
pinned dependencies for Azure integration. And it was pointed
out that we have an open issue for this. Link the same along with
the TODO in the provider.yaml so that if someone takes a look
later, they will know that we have an issue for it and the issue
has relevant description.


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

While working on issue apache#33025, it was unclear on why we've pinned
dependencies for Azure integration. And it was pointed out that
we have an open issue for this. Link the same along with the TODO
in the provider.yaml so that if someone takes a look later, they
will know that we have an issue for it and the issue has relevant
description.
@potiuk potiuk merged commit a069965 into apache:main Aug 3, 2023
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.

2 participants