Skip to content
Snippets Groups Projects

Os path check fix

Merged Os path check fix
os-path-fix into conus-dps
Merged gcorradini requested to merge os-path-fix into conus-dps

A missing "return" statement broke the os_check_path_exists function.

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • gcorradini approved this merge request

    approved this merge request

    By Tempest McCabe on 2023-02-08T15:09:13 (imported from GitLab)

  • gcorradini added <5min label

    added <5min label

  • gcorradini mentioned in commit ade973e9

    mentioned in commit ade973e9

    By Tempest McCabe on 2023-02-08T15:19:08 (imported from GitLab)

  • gcorradini merged

    merged

  • gcorradini
    gcorradini @gcorradini ·
    Author Owner

    I just approved and merged my own MR. In this case was a pretty clear-cut bug fix. In the future though, do we want standards for who reviews/approves?

    By Tempest McCabe on 2023-02-08T15:20:24 (imported from GitLab)

  • gcorradini
    gcorradini @gcorradini ·
    Author Owner

    Probably yes, but with something this small it certainly seems all right with me! Thanks for catching this... literally never noticed it.

    By Eli Orland on 2023-02-08T16:30:10 (imported from GitLab)

Please register or sign in to reply