Skip to content

Conversation

@lonitra
Copy link
Member

@lonitra lonitra commented Jul 26, 2023

Summary of the changes

  • Removes async keyword from method generation when executing 'Generate Async Method' code action to align with C# behavior and update tests
  • Ensure the same namespace that is computed in GenerateMethodCodeActionResolver is used for the code behind file content in tests.

@lonitra lonitra requested a review from a team as a code owner July 26, 2023 20:35
@lonitra lonitra enabled auto-merge (squash) July 26, 2023 21:17
@lonitra lonitra merged commit 48aaedc into dotnet:main Jul 27, 2023
@ghost ghost added this to the Next milestone Jul 27, 2023
@lonitra lonitra deleted the fixasync branch July 27, 2023 15:44
lonitra added a commit to lonitra/razor that referenced this pull request Jul 27, 2023
* Remove async keyword from generate async method

* Ensure namespace for code behind tests is correct
lonitra added a commit that referenced this pull request Jul 27, 2023
…9035)

* Remove async keyword from generate async method

* Ensure namespace for code behind tests is correct
333fred added a commit to 333fred/razor that referenced this pull request Aug 23, 2023
* upstream/main: (188 commits)
  Rename CSHTML file reference. (dotnet#8969)
  Remove Omnisharp logic from main branch (dotnet#9027)
  Update dependencies from https://github.com/dotnet/arcade build 20230726.1
  Fixes CVE-2023-33127 and CVE-2023-33170 (dotnet#9032)
  Remove Async Keyword For Generate Async Method Code Action (dotnet#9030)
  Remove dispatcher from DocumentVersionCache (dotnet#9026)
  Restore perf work. (dotnet#8995)
  Implement priority trigger support
  Change implementations and references
  Rename ProjectSnapshotChangeTrigger and convert to interface
  Updates after merge
  Fix nullability
  Use pattern matching
  Convert to record struct
  Move CloseTextTagOnAutoInsertProvider to FindToken (dotnet#9025)
  Move GenerateMethodCodeActionProvider to FindToken (dotnet#8988)
  Add comment describing when ProjectRazorJson.Version should be incremented
  Some more violations, after the merge
  Remove TryResolveDocument method
  [Infra] 17.8 P1 snap PRs (dotnet#9021)
  ...
@Cosifne Cosifne modified the milestones: Next, 17.8 P3 Sep 25, 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.

5 participants