Replace pdpyras with pagerduty library in PagerDuty integration#48436
Replace pdpyras with pagerduty library in PagerDuty integration#48436simonprydden wants to merge 0 commit intoapache:mainfrom
Conversation
|
Are the changes backward compatible for the users of the provider? |
The migration notes are simple find and replace, except this one quote, which might be breaking as that it the one causing the test to fail. Just trying to debug now
|
|
I think the new library is missing return on the function, see below going to open a ticket as the docstring of the function say there should be response id |
Did you report the issue to the upstream library? is there an issue to track? |
|
OK found it PagerDuty/python-pagerduty#38 |
7aa22fb to
95917f1
Compare
closes #48434
^ 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.rstor{issue_number}.significant.rst, in airflow-core/newsfragments.