Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AIP-84 Get Plugins #43125

Merged
merged 4 commits into from
Oct 22, 2024
Merged

AIP-84 Get Plugins #43125

merged 4 commits into from
Oct 22, 2024

Conversation

pierrejeambrun
Copy link
Member

@pierrejeambrun pierrejeambrun commented Oct 17, 2024

Add get plugins to new AIP.

Some types are list[dict] without more precision at this point (same as the legacy).

But I might explicit these types just in case the front-end need to know about them. (IT is also cleaner for users). done

@pierrejeambrun pierrejeambrun added AIP-84 Modern Rest API legacy api Whether legacy API changes should be allowed in PR labels Oct 17, 2024
@pierrejeambrun pierrejeambrun self-assigned this Oct 17, 2024
@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. labels Oct 17, 2024
Copy link
Contributor

@bbovenzi bbovenzi left a comment

Choose a reason for hiding this comment

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

lgtm after a rebase

@pierrejeambrun pierrejeambrun force-pushed the aip-84-list-plugins branch 2 times, most recently from f04c535 to e178eb0 Compare October 22, 2024 13:16
@pierrejeambrun pierrejeambrun merged commit 35142a9 into apache:main Oct 22, 2024
52 checks passed
@pierrejeambrun pierrejeambrun deleted the aip-84-list-plugins branch October 22, 2024 15:34
harjeevanmaan pushed a commit to harjeevanmaan/airflow that referenced this pull request Oct 23, 2024
* AIP-84 Get Plugins

* Explicit types

* Handle extra unknown properties

* Fix CI
PaulKobow7536 pushed a commit to PaulKobow7536/airflow that referenced this pull request Oct 24, 2024
* AIP-84 Get Plugins

* Explicit types

* Handle extra unknown properties

* Fix CI
ellisms pushed a commit to ellisms/airflow that referenced this pull request Nov 13, 2024
* AIP-84 Get Plugins

* Explicit types

* Handle extra unknown properties

* Fix CI
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
AIP-84 Modern Rest API area:API Airflow's REST/HTTP API area:UI Related to UI/UX. For Frontend Developers. legacy api Whether legacy API changes should be allowed in PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants