updated now

Updated

Updated

Updated

Updated

Updated

Updated

We are checking level status 3 times in same function. So Vinay suggested to use local variable to avoid the inconsistent states across the function.

keep a blank line between declaration(s) and code.

If level can be overridden, you should use get function.

Add blank line before banner.

Added but crucible is not showing this change. Please check bitbucket.

These variables are correctly initialized to the first enum in the list.

Not needed because the retry mechanism is going to be deleted in the next ticket.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

I would rather keep it. Anyways, we need an else block after else if, as per MISRA guidelines.

Not needed because the retry mechanism is going to be deleted in the next ticket.

At end of transition, do you want to initialize the totalSpentChamberFill reset to zero?

Could you use SpentChamberLevel here, instead calling the getLevelStatus function again? Same for next conditions check as well ( line:889)