RFC replace copy_py alias by a function

[mlarose]

TL;DR

The copy_py alias is not recognized in some environments, and the script is unusable as it fails to rsync the desired files.

This change replaces it with a more readily supported bash function

[mlarose]

• RFC replace copy_py alias by a function #21948 👈
• master

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @mlarose and the rest of your teammates on Graphite

[mlarose]

shopt -s expand_alias would need to go as well.

[mlarose]

would replace other instances as well if any, just fishing for alignment here.

[gibsondan]

what environments does it not work in?

can you re-submit with "NO_SKIP" in the commit message so that buildkite actually runs the test that uses this to make sure this works there?

I think i copied this pattern from elsewhere, i definitely did not invent this macro (i think it is used elsewhere)

[mlarose]

what environments does it not work in?

can you re-submit with "NO_SKIP" in the commit message so that buildkite actually runs the test that uses this to make sure this works there?

I think i copied this pattern from elsewhere, i definitely did not invent this macro (i think it is used elsewhere)

macosx ohmyzsh mostly as proposed in the eng onboarding doc - a few plugins and not much else i don't think I have anything fancy going on and after trying 2-3 things, I just decided to replace the alias. it's not the most portable approach vs a bash function. My rationale in not putting any effort to fix my environment is that this is an OSS project and we shouldn't expect all OSS to normalize their environment for something that is easily made cross compatible.