chore: Add healthchecks #11
Annotations
11 warnings and 1 notice
build-and-test
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-dotnet@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
build-and-test:
src/Application/Weather/GetForecast/GetForecastQueryHandler.cs#L15
Dereference of a possibly null reference.
|
build-and-test:
src/Infrastructure/Repositories/WeatherRepository.cs#L16
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
build-and-test:
src/Api/Routes/Weather/Models/GetWeatherResponse.cs#L5
Non-nullable property 'Forecast' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L163
Dereference of a possibly null reference.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L189
Dereference of a possibly null reference.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L166
Public method 'DeleteForecast_AsUser_ShouldReturnForbidden' on test class 'ForecastTests' should be marked as a Fact. Reduce the visibility of the method, or add a Fact attribute to the method. (https://xunit.net/xunit.analyzers/rules/xUnit1013)
|
build-and-test:
src/Application/Weather/GetForecast/GetForecastQueryHandler.cs#L15
Dereference of a possibly null reference.
|
build-and-test:
src/Infrastructure/Repositories/WeatherRepository.cs#L16
This async method lacks 'await' operators and will run synchronously. Consider using the 'await' operator to await non-blocking API calls, or 'await Task.Run(...)' to do CPU-bound work on a background thread.
|
build-and-test:
src/Api/Routes/Weather/Models/GetWeatherResponse.cs#L5
Non-nullable property 'Forecast' must contain a non-null value when exiting constructor. Consider declaring the property as nullable.
|
build-and-test:
tests/Api.E2E/ForecastTests.cs#L163
Dereference of a possibly null reference.
|
6 tests found
There are 6 tests, see "Raw output" for the full list of tests.
|