Azure / Azure/azure-sdk-for-java
OpenRewrite Follow-up Updates
- Dominant language
- Java
- Stars
- 2.6k
- Forks
- 2.2k
- Avg merge
- 1d 22h
- Merged PRs (30d)
- 159
Description
The following updates identified in #44494 need to be made to `azure-openrewrite`:
- [Remove unused plugins](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2108369665)
- [Ditto](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2103912427)
- ~~[Address clientcore RequestContext feature gap (no setHeader api)](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2108352462)~~
- [Follow-up with @srnagar about golden image compiler dependency scopes](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2108368287)
- [Review and update old disabled tests](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2105286147)
- [AccessTokenType can return a NPE](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103921163)
- [Fix AccessTokenType recipe javadoc (package name error)](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103921551)
- [HttpHeaders needs transforms due to dropped Iterable interface](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103935208)
- [Use discrete methods in HttpHeadersCustomRecipe to improve readability](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103938670)
- [BUG: HttpHeadersCustomRecipe setAll recipe does not create HttpHeaders from the map](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103944619)
- [Split and chain visitors in HttpRequestCustomRecipe for readability and maintainence](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2087674505)
- [Split HttpRequestCustomRecipe logic into discrete methods for readability](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103954167)
- [investigate generic type usage in HttpResponse to Response recipe](https://github.com/Azure/azure-sdk-for-java/pull/44494/files#r2103968987)
- [Discussion: Should azure-openrewrite live in the v2 namespace?](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2103898352)
- [Track dropped apis for discussion](https://github.com/Azure/azure-sdk-for-java/pull/44494#discussion_r2087667935)
- Api Change: Look into RequestContext recipes following the recent api change](https://github.com/Azure/azure-sdk-for-java/pull/45534#issuecomment-2923249738)
- Improve Javadocs for all recipes
Contributor guide
Assessment
This issue has not been assessed yet.