Skip to content
This repository was archived by the owner on Aug 27, 2025. It is now read-only.

Update firstTimeInstall.sh source the cprt_bash_aliases from bashrc#25

Open
ErikCald wants to merge 4 commits into
mainfrom
erik-bash-aliases
Open

Update firstTimeInstall.sh source the cprt_bash_aliases from bashrc#25
ErikCald wants to merge 4 commits into
mainfrom
erik-bash-aliases

Conversation

@ErikCald

@ErikCald ErikCald commented Nov 27, 2024

Copy link
Copy Markdown
Contributor

Adds some code into the firstTimeInstall.sh to source our own bash aliases file. Has safety checks that will only put one copy of this code into the bashrc and will only try to source our bash alias file if it exists.

Also created a eaCPRT alias to edit the CPRT alias file.

@ErikCald ErikCald self-assigned this Nov 27, 2024

@ConnorNeed ConnorNeed left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for adding this.

Not sure if we will need it once we get the node management system running since the main aliases we used last year were build, and a handful of launch commands. We have the ./make.sh file for building too. Willing to hear other opinions.

Comment thread firstTimeInstall.sh Outdated
Comment thread cprt_bash_aliases Outdated
@ConnorNeed

Copy link
Copy Markdown
Member

Approving from a implementation standpoint. Up to @ChrisRusu1 as to whether this is a feature we want implemented.

@ChrisRusu1 ChrisRusu1 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can add the specific aliases we used last time later on but I like the idea of having it in the repo so everyones got the same aliases

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants