Skip to content

Merge pull request #142 from bouvet/feat/add-number-of-respondents #72

Merge pull request #142 from bouvet/feat/add-number-of-respondents

Merge pull request #142 from bouvet/feat/add-number-of-respondents #72

Triggered via push January 28, 2025 09:48
Status Failure
Total duration 30d 0h 4m 30s
Artifacts
Build 🛠️
35s
Build 🛠️
Test 🧪
44s
Test 🧪
Deploy/Production
0s
Deploy/Production
Fit to window
Zoom out
Zoom in

Annotations

2 errors and 33 warnings
Deploy to test
WARNING: /home/runner/work/bouvet-developer-survey/bouvet-developer-survey/infra/modules/containerApp.bicep(70,49) : Warning no-hardcoded-env-urls: Environment URLs should not be hardcoded. Use the environment() function to ensure compatibility across clouds. Found this disallowed host: "vault.azure.net" [https://aka.ms/bicep/linter/no-hardcoded-env-urls]
Deploy/Production
The deployment was rejected or didn't satisfy other protection rules.
Build 🛠️
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Build 🛠️: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/TransferObjects/Import/SurveyStructure/SurveyElementBlockDto.cs#L17
Non-nullable property 'Payload' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Build 🛠️: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Structures/BlockElementService.cs#L148
Possible null reference argument for parameter 'source' in 'bool Enumerable.Any<BlockElement>(IEnumerable<BlockElement> source, Func<BlockElement, bool> predicate)'.
Build 🛠️: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Results/ResultService.cs#L191
Possible null reference argument for parameter 'fieldName' in 'void ResultService.CreateNonMultipleValueResponse(List<NewResponseDto> responseDtos, NewResponseDto? existingResponse, string fieldName, string fieldValue, Guid choiceId, Guid answerId)'.
Build 🛠️: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Results/ResultService.cs#L134
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.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/TransferObjects/Import/SurveyStructure/SurveyElementBlockDto.cs#L17
Non-nullable property 'Payload' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Structures/BlockElementService.cs#L148
Possible null reference argument for parameter 'source' in 'bool Enumerable.Any<BlockElement>(IEnumerable<BlockElement> source, Func<BlockElement, bool> predicate)'.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Results/ResultService.cs#L191
Possible null reference argument for parameter 'fieldName' in 'void ResultService.CreateNonMultipleValueResponse(List<NewResponseDto> responseDtos, NewResponseDto? existingResponse, string fieldName, string fieldValue, Guid choiceId, Guid answerId)'.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Results/ResultService.cs#L134
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.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/TransferObjects/Import/SurveyStructure/SurveyElementBlockDto.cs#L17
Non-nullable property 'Payload' must contain a non-null value when exiting constructor. Consider adding the 'required' modifier or declaring the property as nullable.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Structures/BlockElementService.cs#L148
Possible null reference argument for parameter 'source' in 'bool Enumerable.Any<BlockElement>(IEnumerable<BlockElement> source, Func<BlockElement, bool> predicate)'.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Results/ResultService.cs#L191
Possible null reference argument for parameter 'fieldName' in 'void ResultService.CreateNonMultipleValueResponse(List<NewResponseDto> responseDtos, NewResponseDto? existingResponse, string fieldName, string fieldValue, Guid choiceId, Guid answerId)'.
Test 🧪: Bouvet.Developer.Survey.Backend/Bouvet.Developer.Survey.Service/Survey/Results/ResultService.cs#L134
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.
Deploy to test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Deploy to test
WARNING: /home/runner/work/bouvet-developer-survey/bouvet-developer-survey/infra/modules/containerApp.bicep(70,49) : Warning no-hardcoded-env-urls: Environment URLs should not be hardcoded. Use the environment() function to ensure compatibility across clouds. Found this disallowed host: "vault.azure.net" [https://aka.ms/bicep/linter/no-hardcoded-env-urls]