Rename DIR_PREFIX to DIR_SUFFIX

Open heanhsok opened this issue 10 months ago • 1 comments

Follow up from https://github.com/causify-ai/helpers/pull/275/files#r1943494559

We want to rename the variable $DIR_PREFIX to $DIR_SUFFIX to better reflect its actual definition

We can do it after Improve repo_config #88 as this variable exist in multiple python and shell script.

Feb 06 '25 19:02 heanhsok

Agree this can be an easy global replacement. (Ofc any refactoring and / or using links helps avoid repeatition)

Feb 12 '25 14:02 gpsaggese

I have renamed the variable and changed to use the config from our repo_config.yaml where possible in this PR causify-ai/csfy#304
Once we merged this PR, we can replace a few more files (build.py and tmux.py) with links in the parent repo such as //cmamp

FYI @gpsaggese

Apr 15 '25 17:04 heanhsok

Closing as the renaming was done.

Filed https://github.com/causify-ai/csfy/issues/5791 to replace files in symlinks in other repos.

May 04 '25 20:05 heanhsok