значи колега,тои ползва полета така че кода му е ок,не е като да ползва properties,сега дали е правилно е друг въпрос аз лично бих го направил с property и вътре в salary да проверявам годините и така да увеличавам но това е друга тема.  Get И Set се ползват в property не на полетата....Да не говорим че set се ползва доста ситуационно.

Кода изглежда ок проблема е по-скоро в judge Но за всеки случай аз бих сменил името на salary class-a на нещо различно примерно StartUp и направил и него Public и тогава да пробвам отново.Гледам че си махнал namespaces но понякога и тея дребни глупости правят проблем. 

Благодаря за предложенията, но все още не успявам да го подкарам

Ползвам private fields, а не properties, защото по условие се иска да са private и да не могат да се достъпват извън класа освен с метода за увеличаване на заплатата, все пак лекцията е за Енцапсулация. И все пак опитах и с properties, същата грешка, със смяна на името на класа Salary и да стане public, същата грешка, поне да изписваше за какво гърми, а то на details няма никаква информация

ам с промяната на стратегийте това е нормално :Д последните 3 месеца доста пъти се сблъскахме с това,на подготовките за изпитите а задачи от домашните. понякога ни гърмяха с compile time само защото нямахме примерно константи  за магически числа и подобни глупости.Ta най-вероятно judge очаква нещо което го няма и затова си гърми така.Явор най-вероятно я е решил на лекцията може да видиш на видеото.:Д лабс ми бяха скучни и ги прескачах та нямам минало решение в Judge да видя какви са разликите и защо едното минава а другото не.

Tова да имаш property което също може да е private или protected няма нищо лошо :д идеята е че вътре в него си правиш валидацията.. Примерно както си направил метода си там щеше да е this.Salary+=.... а вътре в salary щеше да е : 

protected decimal Salary get=>this.salary; set {

if(this.Age<30) { value+=....}

else {value+=....}

this.salary=value } и общо взето целия метод които си написал става излишен  и горе само ще имаш в main  people.ForEach(p=>p.Salary+=bonus);и толкова.

Нищо не пречи да си имаш private поле.Което да достъпваш с публично property енкапсулацията ти по никакъв начин не е нарушена. 

Току що и аз открих същото, по трудния начин:)
В условието си пише: "You should be able to use the class like this: ... persons.ForEach(p => p.IncreaseSalary(bonus));..." Пропърти Age пък се иска в задача 1, а 2 започва със "Refactor project from last task."