Skip to content

Issue #621: Generate async methods#797

Merged
sjh37 merged 20 commits intosjh37:masterfrom
erwin-faceit:feature/issue-621
Dec 4, 2023
Merged

Issue #621: Generate async methods#797
sjh37 merged 20 commits intosjh37:masterfrom
erwin-faceit:feature/issue-621

Conversation

@erwin-faceit
Copy link
Contributor

@erwin-faceit erwin-faceit commented Mar 13, 2023

As requested in issue #621, I've updated the templates for all frameworks to generate async methods. Your proposal in issue 621 kind of works, but generates a return model for all stored procedures.

This PR will generate:

  • Async methods for all stored procedures which have a single return type,
  • Add CancellationToken arguments to all generated stored procedures (with default value)
  • Invoke ExecuteSqlCommandAsync or ExecuteSqlRawAsync (depending on the framework), both have cancellation support

I have tested this with EF6 and EF Core 3 myself, for our own projects.

I've also used the Tester projects to generate the provided contexts again, but I noticed a lot of these projects will not compile after regeneration (unrelated to this PR, this is also the case with the current 3.8.0 and 3.8.1 tagged versions).

@dsfr-be
Copy link
Contributor

dsfr-be commented Nov 29, 2023

Hi,

Is there a timeline when this PR will make it to master?

KR

@sjh37 sjh37 closed this Dec 4, 2023
@sjh37 sjh37 reopened this Dec 4, 2023
@sjh37 sjh37 merged commit 0b7e5bf into sjh37:master Dec 4, 2023
sjh37 added a commit that referenced this pull request Dec 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants