azure-sdk-for-net icon indicating copy to clipboard operation
azure-sdk-for-net copied to clipboard

ClientModel: Migrate stable types from Azure.Core Shared source to public types

Open christothes opened this issue 3 years ago • 12 comments
trafficstars

Library name

Azure.Core

Types

Note: shared source in the autorest.csharp repo is now tracked by https://github.com/Azure/azure-sdk-for-net/issues/32006

christothes avatar Apr 25 '22 15:04 christothes

Includes migrating Generator Shared source to Azure.Core public types as well.

Related to:

  • https://github.com/Azure/autorest.csharp/issues/2231
  • https://github.com/Azure/autorest.csharp/issues/2148

In the Zinc timeframe, we plan to make a plan for this migration.

annelo-msft avatar Aug 24 '22 23:08 annelo-msft

Related to https://github.com/Azure/azure-sdk-for-net/issues/30714

annelo-msft avatar Aug 24 '22 23:08 annelo-msft

As @christothes brought up in triage, perhaps this could be broken up into bite-sized issues - some of which could be done in MQ potentially.

heaths avatar Oct 25 '22 17:10 heaths

Specific customer ask for MultipartFormDataContent to be made public in https://github.com/Azure/azure-sdk-for-net/issues/24819

annelo-msft avatar May 11 '23 19:05 annelo-msft

Consolidating types that standardize exceptions to a single type could address the following files:

annelo-msft avatar Aug 08 '23 15:08 annelo-msft

We have had a few issues where we have considered having Helper\Utility APIs that may be useful to other SDKs and we should consider exposing those APIs are public helper methods in core.

pallavit avatar Dec 18 '23 17:12 pallavit

We have had a few issues where we have considered having Helper\Utility APIs that may be useful to other SDKs and we should consider exposing those APIs are public helper methods in core.

@pallavit, would it be possible to link those issues to this one so we can track them together?

annelo-msft avatar Jan 03 '24 01:01 annelo-msft

Why is this a P1? For instance, why supporting this is higher priority than enabling support for pagination in debranded toolchain?

pallavit avatar Mar 12 '24 23:03 pallavit

Why is this a P1? For instance, why supporting this is higher priority than enabling support for pagination in debranded toolchain?

My initial impression is that for our efforts to parallelize Core work, it'd be good to have a plan here to unblock forward progress as we chip away on these types. For pagination, I don't know of a concrete scenario, but I'm open to reprioritization based on new information, so please view what we have today as the start of the conversation and happy to reshuffle as needed!

annelo-msft avatar Mar 12 '24 23:03 annelo-msft

TODO: consider adding a helper to polyfill missing APIs on netstandard2.0 in serialization routines, e.g. to use one type for instances of https://github.com/search?q=repo%3AAzure%2Fazure-sdk-for-net+NET6_0_OR_GREATER+NOT+path%3AResourceManager&type=code&p=3

annelo-msft avatar Mar 19 '24 22:03 annelo-msft

Hi @christothes, we deeply appreciate your input into this project. Regrettably, this issue has remained inactive for over 2 years, leading us to the decision to close it. We've implemented this policy to maintain the relevance of our issue queue and facilitate easier navigation for new contributors. If you still believe this topic requires attention, please feel free to create a new issue, referencing this one. Thank you for your understanding and ongoing support.

github-actions[bot] avatar Apr 24 '24 18:04 github-actions[bot]

Issue still active and slated for this semester

annelo-msft avatar Apr 24 '24 20:04 annelo-msft

Issue still active and slated for this semester

After 30 days of no activity, the bot will close this again. I'm going to clone the issue in order to buy us two more years of peace.

jsquire avatar Apr 29 '24 14:04 jsquire

New issue: # #43760

jsquire avatar Apr 29 '24 14:04 jsquire