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

Add OpenWithCreate(...) as a proof of concept. #4162

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

mitchdenny
Copy link
Member

@mitchdenny mitchdenny commented May 13, 2024

Related to #4160

Microsoft Reviewers: Open in CodeFlow

@dotnet-issue-labeler dotnet-issue-labeler bot added the area-components Issues pertaining to Aspire Component packages label May 13, 2024
@mitchdenny mitchdenny added this to the Backlog milestone May 13, 2024
@mitchdenny mitchdenny self-assigned this May 13, 2024
/// the original exception is thrown.
/// </para>
/// </remarks>
public static void OpenWithCreate(this SqlConnection connection)
Copy link
Member

Choose a reason for hiding this comment

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

I mean this isn't public API we would expose right? This should be an ADO.NET thing.

Copy link
Member

Choose a reason for hiding this comment

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

I said the same in #4160 (comment).

(not sure where the conversation should happen - the issue or the PR)

Copy link
Member Author

Choose a reason for hiding this comment

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

Do it in the issue. This was just so we had something concrete to talk about.

And I agree this should be something in ADO.NET

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-components Issues pertaining to Aspire Component packages
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants