dotnet / aspire-samples

MIT License
571 stars 158 forks source link

Fix incorrect execution strategy name #289

Closed hognevevle closed 1 month ago

AndriySvyryd commented 1 month ago

The name is already correct. Though we might consider renaming it to something less redundant, @DamianEdwards

hognevevle commented 1 month ago

@AndriySvyryd That's weird, because only after my proposed change does the code compile. CleanShot 2024-06-24 at 22 12 16

With the original code: CleanShot 2024-06-24 at 22 13 05

AndriySvyryd commented 1 month ago

It's linked in the project https://github.com/hognevevle/aspire-samples/blob/main/samples/DatabaseMigrations/DatabaseMigrations.MigrationService/DatabaseMigrations.MigrationService.csproj#L12 Perhaps an IDE issue?

hognevevle commented 1 month ago

Sorry, of course you are right, I didn't realize there was an override in play here. It's was my own code not compiling, not the sample repo, so I was of course missing that file. 🫣