fwcommon

Clone Tools
  • last updated a few seconds ago
Constraints
Constraints: committers
 
Constraints: files
Constraints: dates
Merge branch 'staging' into DEN-15246-active-reservoir-empty-post-concentrate-alarm-recovery

DEN-15246 updated the conductivity sensors calibration table

Done

Done

Is this a big risk? At this time I am not too concerned about it. If you think it should be addressed, I can make changes.

Is this a big risk? At this time I am not too concerned about it. If you think it should be addressed, I can make changes.

Done

Done

Done

Done

This is 105.

This is 105.

DEN-15274 added HD test configurations

DG-DEN-15274_HD DG Test Configuration Setup
DG-DEN-15274_HD DG Test Configuration Setup
DEN-15274 renamed the DG test configurations

Bamboo Commit: Updated the Copyright section and replaced tabs with 4 spaces

Merged DEN-15199

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Done

Done

Done

Done

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Merge branch 'DEN-15199-fw_dn_sprint_95-part-2' into develop

This was updated but not pushed.

This was updated but not pushed.

This was a mistake. I fixed it.

This was a mistake. I fixed it.

Merge branch 'staging' into DEN-15199-fw_dn_sprint_95-part-2

DEN-15199 addressed the code review comments

Is this needed? Why would a worn cartridge fail pressure test?

Is this needed? Why would a worn cartridge fail pressure test?

Why isn't this just an else? And if else should be swapped so normal action (alarm) comes first with if and the else should be the exception (else).

Why isn't this just an else? And if else should be swapped so normal action (alarm) comes first with if and the else should be the exception (else).

I don't think a worn cartridge should fail here (should still be dry). I think only wet cartridge should be checked here.

I don't think a worn cartridge should fail here (should still be dry). I think only wet cartridge should be checked here.

RESOLVED in CODE WALKTHROUGH.

RESOLVED in CODE WALKTHROUGH.

Recommend being more direct in naming: TEST_CONFIG_USE_WET_CARTRIDGE TEST_CONFIG_USE_WORN_CARTRIDGE

Recommend being more direct in naming:
TEST_CONFIG_USE_WET_CARTRIDGE
TEST_CONFIG_USE_WORN_CARTRIDGE