Skip to content

fix(effort): correct harvest custodial dept and exclude emeritus faculty#232

Open
rlorenzo wants to merge 1 commit into
mainfrom
effort-harvest-issues
Open

fix(effort): correct harvest custodial dept and exclude emeritus faculty#232
rlorenzo wants to merge 1 commit into
mainfrom
effort-harvest-issues

fix(effort): correct harvest custodial dept and exclude emeritus faculty

3388b13
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / CodeQL succeeded Jun 17, 2026 in 2s

1 new alert

New alerts in code changed by this pull request

  • 1 note

See annotations below for details.

View all branch alerts.

Annotations

Check notice on line 412 in web/Areas/Effort/Services/Harvest/NonCrestHarvestPhase.cs

See this annotation in the file changed.

Code scanning / CodeQL

Missed opportunity to use Where Note

This foreach loop
implicitly filters its target sequence
- consider filtering the sequence explicitly using '.Where(...)'.