Skip to content

Merge pull request #140 from bouvet/feat/add-authentication #65

Merge pull request #140 from bouvet/feat/add-authentication

Merge pull request #140 from bouvet/feat/add-authentication #65

Triggered via push January 23, 2025 09:50
Status Success
Total duration 2h 52m 38s
Artifacts
Fit to window
Zoom out
Zoom in

Deployment protection rules

Reviewers, timers, and other rules protecting deployments in this run
Event Environments Comment
asleboon
approved Jan 23, 2025
Production

Annotations

2 errors and 36 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
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]
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/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 🧪
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
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/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]
Deploy/Production
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Deploy/Production
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]