Fix clone with relative restart paths

payu-org / payu

A workflow management tool for numerical models on the NCI computing systems

Apache License 2.0

21 stars 27 forks source link

Fix clone with relative restart paths #516

Closed jo-basevi closed 1 month ago

jo-basevi commented 2 months ago

Added a fix for payu clone -r to resolve relative restart paths - with respect to the directory where payu clone is run from. I've changed it to call the helper method that checks restart path exists and resolves to an absolute path, before changing directory to the cloned repository which runs the checkout step.

Closes #514

pep8speaks commented 2 months ago

Hello @jo-basevi! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

There are currently no PEP 8 issues detected in this Pull Request. Cheers! :beers:

Comment last updated at 2024-09-25 23:21:02 UTC

coveralls commented 2 months ago

coverage: 58.488% (+0.2%) from 58.285% when pulling 2c48e3cafa4f43692054b9041dc29765fd101fe3 on ACCESS-NRI:514-fix-payu-clone-relative-restart-paths into f4af1ffaf49381a8a789faf77d75ee19ab89fb8e on payu-org:master.

jo-basevi commented 2 months ago

As there were merge conflicts, I rebased the branch onto latest changes on the main