Skip to content

fix: move bashrc sourcing to bottom of initrc so bashrc can override …#2523

Merged
RohitKushvaha01 merged 2 commits into
Acode-Foundation:mainfrom
RohitKushvaha01:fix/initrc-bashrc-order
Jul 22, 2026
Merged

fix: move bashrc sourcing to bottom of initrc so bashrc can override …#2523
RohitKushvaha01 merged 2 commits into
Acode-Foundation:mainfrom
RohitKushvaha01:fix/initrc-bashrc-order

Conversation

@RohitKushvaha01

Copy link
Copy Markdown
Member

…initrc settings

@greptile-apps

greptile-apps Bot commented Jul 22, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

This PR moves Bash configuration sourcing to the end of the Alpine terminal init file. The main changes are:

  • Sources configuration after terminal defaults are defined.
  • Expands the intended override scope beyond PROMPT_COMMAND.

Confidence Score: 4/5

The configuration precedence issue should be fixed before merging.

  • The system Bash configuration runs after the user configuration.
  • Conflicting system settings can replace the user overrides this change intends to preserve.

src/plugins/terminal/scripts/init-alpine.sh

Important Files Changed

Filename Overview
src/plugins/terminal/scripts/init-alpine.sh Moves Bash configuration sourcing after generated defaults, but leaves the system file with final precedence over user settings.

Reviews (1): Last reviewed commit: "fix: move bashrc sourcing to bottom of i..." | Re-trigger Greptile

Comment thread src/plugins/terminal/scripts/init-alpine.sh Outdated
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
@RohitKushvaha01
RohitKushvaha01 added this pull request to the merge queue Jul 22, 2026
Merged via the queue into Acode-Foundation:main with commit fad7572 Jul 22, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from Backlog to Done in The Code Board - Acode Jul 22, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant