New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Scheduler and plannings management (#3255) #3256

Draft

kbeaugrand wants to merge 1 commit into main from main-vnext

Draft

Scheduler and plannings management (#3255) #3256

Scheduler and plannings management (#3255)

GitHub Advanced Security / CodeQL succeeded Dec 5, 2024 in 8s

11 new alerts

New alerts in code changed by this pull request

5 warnings
6 notes

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 68 in src/IoTHub.Portal.Client/Dialogs/Planning/DeletePlanningDialog.razor

Code scanning / CodeQL

Missed opportunity to use Select Note

This foreach loop immediately

maps its iteration variable to another variable

- consider mapping the sequence explicitly using '.Select(...)'.

Check notice on line 108 in src/IoTHub.Portal.Infrastructure/Jobs/SendPlanningCommandJob.cs

Code scanning / CodeQL

Generic catch clause Note

Generic catch clause.

Check notice on line 123 in src/IoTHub.Portal.Infrastructure/Jobs/SendPlanningCommandJob.cs

Code scanning / CodeQL

Generic catch clause Note

Generic catch clause.

Check notice on line 131 in src/IoTHub.Portal.Infrastructure/Jobs/SendPlanningCommandJob.cs

Code scanning / CodeQL

Missed opportunity to use Where Note

This foreach loop

implicitly filters its target sequence

- consider filtering the sequence explicitly using '.Where(...)'.

Check notice on line 169 in src/IoTHub.Portal.Infrastructure/Jobs/SendPlanningCommandJob.cs

Code scanning / CodeQL

Missed opportunity to use Where Note

This foreach loop

implicitly filters its target sequence

- consider filtering the sequence explicitly using '.Where(...)'.

Check notice on line 195 in src/IoTHub.Portal.Infrastructure/Jobs/SendPlanningCommandJob.cs

Code scanning / CodeQL

Missed opportunity to use Where Note

This foreach loop

implicitly filters its target sequence

- consider filtering the sequence explicitly using '.Where(...)'.

Check warning on line 566 in src/IoTHub.Portal.Tests.Unit/Client/Components/Planning/EditPlanningTest.cs

Code scanning / CodeQL

Useless assignment to local variable Warning

This assignment to

deviceModels