Merge eshop with aspire preview5 #15
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Add public get methods for GetOrderDate and GetDescription * Make query more idiomatic As mentioned in issue #23 * Remove unused MapOrderItems * Make CardTypesQuery more idomatic * Make buyerId a auto property Make buyerId a auto property in oder to use buyerid inside linq queries. * GetOrdersFromUserAsync to linq Make GetOrdersFromUserAsync more idomatic and remove unused NpgsqlDataSource * - refactor: converted private fields to properties on Order and OrderItem - don't use repository any more * Add buyer navigation property * Introduce SetPaymentMethodVerified method on Order to better align with business operations * Convert GetTotal to expression body and remove unnecessary HasColumnName. * Merge enum Orderstatus changes * Remove explicit config and Add private setters so implicit configuration from ef core will work
* Update to aspire preview3 * Make preview3 work * Remove container image base * Name fixes * Fix connections * Remove placeholder uri logic * Work around SDK bug * Fix images * Use ContainerRepository * Fixed registry * Fixed jwt keys * Disable key rotation for now * Made it work - Added forwarded headers setting * Remove warning suppression from Aspire components. dotnet/runtime#94117 has been fixed and flown to the Aspire components. They no longer produce AOT warnings. * - Remove temporary nuget.config - Update to official Aspire preview3 - Fix up some Extensions code to conform to latest template * Remove unnecessary .sln file * Remove ContainerBaseImage change. --------- Co-authored-by: David Fowler <[email protected]> Co-authored-by: Eric Erhardt <[email protected]> Co-authored-by: Aditya Mandaleeka <[email protected]>
Bumps [xunit](https:/xunit/xunit) from 2.5.1 to 2.6.6. - [Commits](xunit/xunit@2.5.1...2.6.6) --- updated-dependencies: - dependency-name: xunit dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [xunit.runner.visualstudio](https:/xunit/visualstudio.xunit) from 2.5.1 to 2.5.6. - [Release notes](https:/xunit/visualstudio.xunit/releases) - [Commits](xunit/visualstudio.xunit@2.5.1...2.5.6) --- updated-dependencies: - dependency-name: xunit.runner.visualstudio dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…7.1 (#190) Bumps [OpenTelemetry.Instrumentation.AspNetCore](https:/open-telemetry/opentelemetry-dotnet) from 1.6.0-beta.2 to 1.7.1. - [Release notes](https:/open-telemetry/opentelemetry-dotnet/releases) - [Commits](open-telemetry/opentelemetry-dotnet@1.6.0-beta.2...Instrumentation.Http-1.7.1) --- updated-dependencies: - dependency-name: OpenTelemetry.Instrumentation.AspNetCore dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…185) Bumps [OpenTelemetry.Instrumentation.Http](https:/open-telemetry/opentelemetry-dotnet) from 1.6.0-beta.2 to 1.7.1. - [Release notes](https:/open-telemetry/opentelemetry-dotnet/releases) - [Commits](open-telemetry/opentelemetry-dotnet@1.6.0-beta.2...Instrumentation.Http-1.7.1) --- updated-dependencies: - dependency-name: OpenTelemetry.Instrumentation.Http dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps `MicrosoftExtensionsVersion` from 8.0.0 to 8.2.0. Updates `Microsoft.Extensions.Http.Resilience` from 8.0.0 to 8.2.0 - [Release notes](https:/dotnet/extensions/releases) - [Commits](https:/dotnet/extensions/commits) Updates `Microsoft.Extensions.Options` from 8.0.0 to 8.0.2 - [Release notes](https:/dotnet/runtime/releases) - [Commits](dotnet/runtime@v8.0.0...v8.0.2) --- updated-dependencies: - dependency-name: Microsoft.Extensions.Http.Resilience dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: Microsoft.Extensions.Options dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [OpenTelemetry.Exporter.OpenTelemetryProtocol](https:/open-telemetry/opentelemetry-dotnet) from 1.7.0-alpha.1 to 1.7.0. - [Release notes](https:/open-telemetry/opentelemetry-dotnet/releases) - [Commits](open-telemetry/opentelemetry-dotnet@core-1.7.0-alpha.1...core-1.7.0) --- updated-dependencies: - dependency-name: OpenTelemetry.Exporter.OpenTelemetryProtocol dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [OpenTelemetry.Instrumentation.Runtime](https:/open-telemetry/opentelemetry-dotnet-contrib) from 1.5.1 to 1.7.0. - [Release notes](https:/open-telemetry/opentelemetry-dotnet-contrib/releases) - [Commits](open-telemetry/opentelemetry-dotnet-contrib@Exporter.Geneva-1.5.1...Exporter.Geneva-1.7.0) --- updated-dependencies: - dependency-name: OpenTelemetry.Instrumentation.Runtime dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [OpenTelemetry.Instrumentation.GrpcNetClient](https:/open-telemetry/opentelemetry-dotnet) from 1.6.0-beta.2 to 1.6.0-beta.3. - [Release notes](https:/open-telemetry/opentelemetry-dotnet/releases) - [Commits](open-telemetry/opentelemetry-dotnet@1.6.0-beta.2...1.6.0-beta.3) --- updated-dependencies: - dependency-name: OpenTelemetry.Instrumentation.GrpcNetClient dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Microsoft.NET.Test.Sdk](https:/microsoft/vstest) from 17.7.2 to 17.9.0. - [Release notes](https:/microsoft/vstest/releases) - [Changelog](https:/microsoft/vstest/blob/main/docs/releases.md) - [Commits](microsoft/vstest@v17.7.2...v17.9.0) --- updated-dependencies: - dependency-name: Microsoft.NET.Test.Sdk dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps `AspnetVersion` from 8.0.1 to 8.0.2. Updates `Microsoft.AspNetCore.OpenApi` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Authentication.JwtBearer` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Identity.EntityFrameworkCore` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.Extensions.Identity.Stores` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Identity.UI` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Authentication.OpenIdConnect` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Components.Web` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Components.QuickGrid` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.Mvc.Testing` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) Updates `Microsoft.AspNetCore.TestHost` from 8.0.1 to 8.0.2 - [Release notes](https:/dotnet/aspnetcore/releases) - [Changelog](https:/dotnet/aspnetcore/blob/main/docs/ReleasePlanning.md) - [Commits](dotnet/aspnetcore@v8.0.1...v8.0.2) --- updated-dependencies: - dependency-name: Microsoft.AspNetCore.OpenApi dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Authentication.JwtBearer dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Identity.EntityFrameworkCore dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.Extensions.Identity.Stores dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Identity.UI dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Authentication.OpenIdConnect dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Components.Web dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Components.QuickGrid dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.Mvc.Testing dependency-type: direct:production update-type: version-update:semver-patch - dependency-name: Microsoft.AspNetCore.TestHost dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Npgsql.EntityFrameworkCore.PostgreSQL](https:/npgsql/efcore.pg) from 8.0.0 to 8.0.2. - [Release notes](https:/npgsql/efcore.pg/releases) - [Commits](npgsql/efcore.pg@v8.0.0...v8.0.2) --- updated-dependencies: - dependency-name: Npgsql.EntityFrameworkCore.PostgreSQL dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [OpenTelemetry.Extensions.Hosting](https:/open-telemetry/opentelemetry-dotnet) from 1.7.0-alpha.1 to 1.7.0. - [Release notes](https:/open-telemetry/opentelemetry-dotnet/releases) - [Commits](open-telemetry/opentelemetry-dotnet@core-1.7.0-alpha.1...core-1.7.0) --- updated-dependencies: - dependency-name: OpenTelemetry.Extensions.Hosting dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Dapper](https:/DapperLib/Dapper) from 2.0.151 to 2.1.28. - [Release notes](https:/DapperLib/Dapper/releases) - [Commits](DapperLib/Dapper@2.0.151...2.1.28) --- updated-dependencies: - dependency-name: Dapper dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Microsoft.EntityFrameworkCore.Tools](https:/dotnet/efcore) from 8.0.0 to 8.0.2. - [Release notes](https:/dotnet/efcore/releases) - [Commits](dotnet/efcore@v8.0.0...v8.0.2) --- updated-dependencies: - dependency-name: Microsoft.EntityFrameworkCore.Tools dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [AspNetCore.HealthChecks.Uris](https:/Xabaril/AspNetCore.Diagnostics.HealthChecks) from 7.0.0 to 8.0.0. - [Release notes](https:/Xabaril/AspNetCore.Diagnostics.HealthChecks/releases) - [Changelog](https:/Xabaril/AspNetCore.Diagnostics.HealthChecks/blob/master/doc/ui-changelog.md) - [Commits](Xabaril/AspNetCore.Diagnostics.HealthChecks@preview-all-7.0.0...release-ui-8.0.0) --- updated-dependencies: - dependency-name: AspNetCore.HealthChecks.Uris dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [Polly](https:/App-vNext/Polly) from 8.2.0 to 8.3.0. - [Release notes](https:/App-vNext/Polly/releases) - [Changelog](https:/App-vNext/Polly/blob/main/CHANGELOG.md) - [Commits](App-vNext/Polly@8.2.0...8.3.0) --- updated-dependencies: - dependency-name: Polly dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…ityFramework.Storage, Duende.IdentityServer.Storage and Duende.IdentityServer (#208) Bumps [Duende.IdentityServer.EntityFramework](https:/DuendeSoftware/IdentityServer), [Duende.IdentityServer.EntityFramework.Storage](https:/DuendeSoftware/IdentityServer), [Duende.IdentityServer.Storage](https:/DuendeSoftware/IdentityServer) and [Duende.IdentityServer](https:/DuendeSoftware/IdentityServer). These dependencies needed to be updated together. Updates `Duende.IdentityServer.EntityFramework` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) Updates `Duende.IdentityServer.EntityFramework.Storage` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) Updates `Duende.IdentityServer.Storage` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) Updates `Duende.IdentityServer` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) --- updated-dependencies: - dependency-name: Duende.IdentityServer.EntityFramework dependency-type: direct:production update-type: version-update:semver-major - dependency-name: Duende.IdentityServer.EntityFramework.Storage dependency-type: direct:production update-type: version-update:semver-major - dependency-name: Duende.IdentityServer.Storage dependency-type: direct:production update-type: version-update:semver-major - dependency-name: Duende.IdentityServer dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
…Duende.IdentityServer.Storage (#215) Bumps [Duende.IdentityServer.AspNetIdentity](https:/DuendeSoftware/IdentityServer), [Duende.IdentityServer](https:/DuendeSoftware/IdentityServer) and [Duende.IdentityServer.Storage](https:/DuendeSoftware/IdentityServer). These dependencies needed to be updated together. Updates `Duende.IdentityServer.AspNetIdentity` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) Updates `Duende.IdentityServer` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) Updates `Duende.IdentityServer.Storage` from 6.3.5 to 7.0.1 - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@6.3.5...7.0.1) --- updated-dependencies: - dependency-name: Duende.IdentityServer.AspNetIdentity dependency-type: direct:production update-type: version-update:semver-major - dependency-name: Duende.IdentityServer dependency-type: direct:production update-type: version-update:semver-major - dependency-name: Duende.IdentityServer.Storage dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Moseley <[email protected]>
* Bump GrpcVersion from 2.59.0 to 2.61.0 Bumps `GrpcVersion` from 2.59.0 to 2.61.0. Updates `Grpc.AspNetCore` from 2.59.0 to 2.61.0 - [Release notes](https:/grpc/grpc-dotnet/releases) - [Changelog](https:/grpc/grpc-dotnet/blob/master/doc/release_process.md) - [Commits](grpc/grpc-dotnet@v2.59.0...v2.61.0) Updates `Grpc.AspNetCore.Server.ClientFactory` from 2.59.0 to 2.61.0 - [Release notes](https:/grpc/grpc-dotnet/releases) - [Changelog](https:/grpc/grpc-dotnet/blob/master/doc/release_process.md) - [Commits](grpc/grpc-dotnet@v2.59.0...v2.61.0) Updates `Grpc.Tools` from 2.59.0 to 2.61.0 - [Release notes](https:/grpc/grpc/releases) - [Changelog](https:/grpc/grpc/blob/master/doc/grpc_release_schedule.md) - [Commits](https:/grpc/grpc/commits) --- updated-dependencies: - dependency-name: Grpc.AspNetCore dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: Grpc.AspNetCore.Server.ClientFactory dependency-type: direct:production update-type: version-update:semver-minor - dependency-name: Grpc.Tools dependency-type: direct:production update-type: version-update:semver-minor ... Signed-off-by: dependabot[bot] <[email protected]> * fix --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: Dan Moseley <[email protected]>
Move deserialization of event bus message outside of for each loop
* Replaced Map forwarder with Yarp reverse Proxy for mobile BFF * Update YARP config * Update port --------- Co-authored-by: Srushti Pasari94 <[email protected]> Co-authored-by: James Montemagno <[email protected]>
…and AI chat window size fixed (#236) * Updated CSS for good readability of the item number of shopping bags * fixed the partially obscured AI chat window --------- Co-authored-by: Srushti Pasari94 <[email protected]>
Bumps [Microsoft.SemanticKernel](https:/microsoft/semantic-kernel) from 1.6.2 to 1.6.3. - [Release notes](https:/microsoft/semantic-kernel/releases) - [Commits](microsoft/semantic-kernel@dotnet-1.6.2...dotnet-1.6.3) --- updated-dependencies: - dependency-name: Microsoft.SemanticKernel dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* azd instructions * lint * lint
* Bump NSubstitute.Analyzers.CSharp from 1.0.15 to 1.0.17 Bumps [NSubstitute.Analyzers.CSharp](https:/nsubstitute/NSubstitute.Analyzers) from 1.0.15 to 1.0.17. - [Changelog](https:/nsubstitute/NSubstitute.Analyzers/blob/1.0.17/ReleaseNotes.md) - [Commits](nsubstitute/NSubstitute.Analyzers@1.0.15...1.0.17) --- updated-dependencies: - dependency-name: NSubstitute.Analyzers.CSharp dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> * cleanup test * try fix test * task again with order * from docs * typo * disable warning for now --------- Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Co-authored-by: James Montemagno <[email protected]>
* Add .NET MAUI CI * cleanup * add ignores * updae installes * add no warn
The Kernel was never registered in DI, so it wasn't possible to use the AI services.
Bumps [Duende.IdentityServer.Storage](https:/DuendeSoftware/IdentityServer) from 7.0.3 to 7.0.4. - [Release notes](https:/DuendeSoftware/IdentityServer/releases) - [Commits](DuendeSoftware/IdentityServer@7.0.3...7.0.4) --- updated-dependencies: - dependency-name: Duende.IdentityServer.Storage dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <[email protected]> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
BrennanConroy
force-pushed
the
brecon/aprMerge
branch
from
2b5804d
to
cd453ab
Compare
BrennanConroy
force-pushed
the
brecon/aprMerge
branch
from
cd453ab
to
afd500c
Compare
adityamandaleeka
approved these changes
Apr 17, 2024
|
Let's disable playwright tests in this repo for now while we figure out what to do about it. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Majority of changes are in the latest commit(s)
afd500c