Closed thangckt closed 1 month ago
dpdispatcher/machines/pbs.py
[warning] 217-217: dpdispatcher/machines/pbs.py#L217 Added line #L217 was not covered by tests --- [warning] 220-221: dpdispatcher/machines/pbs.py#L220-L221 Added lines #L220 - L221 were not covered by tests
dpdispatcher/machines/pbs.py (1)
`319-319`: **Verify the correctness of the new argument.** The new argument `job_name` is defined as an optional string with a default value of "wDPjob". Ensure that this logic is correct and that the argument is properly used in the codebase.
Attention: Patch coverage is 25.00000%
with 3 lines
in your changes missing coverage. Please review.
Project coverage is 47.48%. Comparing base (
dfa143d
) to head (9b1c566
).
Files | Patch % | Lines |
---|---|---|
dpdispatcher/machines/pbs.py | 25.00% | 3 Missing :warning: |
:exclamation: There is a different number of reports uploaded between BASE (dfa143d) and HEAD (9b1c566). Click for more details.
HEAD has 1 upload less than BASE
| Flag | BASE (dfa143d) | HEAD (9b1c566) | |------|------|------| ||15|14|
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@njzjz
add option to set
job_name
in SGE systemSummary by CodeRabbit