adding notification for which directory cloned into

[cwhitlock-NOAA]

No description provided.

[cwhitlock-NOAA]

Yes, it's just the print statement. It's a hello world for understanding the merge process.

[bcc2761]

@cwhitlock-NOAA Sorry I didn't get back to you on Friday; I actually worked on a fix for this as soon as I saw your issue submitted, but I still want to get your merge in. Would you mind making a new commit with this change?

bcc2761@e48e227

Click prefers click.echo over print (reference: https://click.palletsprojects.com/en/8.1.x/quickstart/#echoing), and I thought that moving this notification to the end would be better than before the cloning occurs. Let me know if you have any thoughts

[cwhitlock-NOAA]

Thank you for the heads-up about click.echo! I do have a quibble about the timing of the clone notification: If you don't print out the full path of where the clone is taking place before the clone command, the error from a failed clone command only prints the last two elements of the directory path (i.e. the experiment directory and the clone dir). I added that print, in part, because I spent an embarrassing amount of time confusing the cylc-src and cylc-run directories a few weeks back when I was trying to clean up an experiment. I don't think this fix is going to prevent users from getting foggy at 4 PM on a Friday, but printing the full path should make it easier to figure out what they are doing wrong.

…

On Mon, Mar 4, 2024 at 11:32 AM Bennett Chang ***@***.***> wrote: @cwhitlock-NOAA <https://github.com/cwhitlock-NOAA> Sorry I didn't get back to you on Friday; I actually worked on a fix for this as soon as I saw your issue submitted, but I still want to get your merge in. Would you mind making a new commit with this change? ***@***.*** <bcc2761@e48e227> Click prefers click.echo over print (reference: https://click.palletsprojects.com/en/8.1.x/quickstart/#echoing), and I thought that moving this notification to the end would be better than before the cloning occurs. Let me know if you have any thoughts — Reply to this email directly, view it on GitHub <#42 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AC362WXEUNNKEGMYZHU3XN3YWSOZRAVCNFSM6AAAAABECMC2VWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTSNZWHE4TINRUHA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

-- Carolyn Whitlock

[bcc2761]

Good point. I didn't account for failed clones that might confuse the user. Yeah, we can add it before the clone occurs then