Skip to content

AIP-84 Get Plugins#43125

Merged
pierrejeambrun merged 4 commits intoapache:mainfrom
astronomer:aip-84-list-plugins
Oct 22, 2024
Merged

AIP-84 Get Plugins#43125
pierrejeambrun merged 4 commits intoapache:mainfrom
astronomer:aip-84-list-plugins

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
@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