Skip to content

Conversation

@bugraoz93
Copy link
Contributor

@bugraoz93 bugraoz93 commented Nov 4, 2024

Related: #43496
The endpoint in the tests is deleted in the related PR. Deleting them should fix CI test issues. This is blocking the PRs too.


^ 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.

@bugraoz93
Copy link
Contributor Author

@dstandish I think this endpoint is deleted in #43496. Deleting them should fix CI test issues but I wanted to double-check with you. I am not sure how we are going to handle the permissions and tests in the FastAPI so maybe deleting them would be a good idea for now.

@bugraoz93
Copy link
Contributor Author

I think mine is a duplicate of #43649. I am closing this one.

@bugraoz93
Copy link
Contributor Author

Thanks a lot for a swift review @jscheffl! 🙏

@bugraoz93 bugraoz93 closed this Nov 4, 2024
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