Skip to content

chore: remove starship config#128

Draft
renner0e wants to merge 2 commits intomainfrom
remove-starship
Draft

chore: remove starship config#128
renner0e wants to merge 2 commits intomainfrom
remove-starship

Conversation

@renner0e
Copy link
Copy Markdown
Member

@renner0e renner0e commented Mar 7, 2026

If we no longer ship it on the image then I don't see a reason to include the config for this.

The profile.d script can stay in my opinion to make the migration
easier.

related: ublue-os/aurora#1742

If we no longer ship it on the image then I don't see a reason to
include the config for this.

The profile.d script can stay in my opinion to make the migration
easier.

related: ublue-os/aurora#1742
@codacy-production
Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

TIP This summary will be updated as you push new changes. Give us feedback

@T045T
Copy link
Copy Markdown

T045T commented May 6, 2026

The profile.d script can stay in my opinion to make the migration easier.

Not sure that that's a good idea. On the current release of Aurora, starship isn't present, and the profile.d script means that every time I open a terminal, I get an error message. Granted, this isn't on a fresh install (and on one rebased from Kinoite, back when that was still half-supported), but I imagine quite a few other users will see that same error.

@renner0e
Copy link
Copy Markdown
Member Author

renner0e commented May 6, 2026

The profile.d script can stay in my opinion to make the migration easier.

Not sure that that's a good idea. On the current release of Aurora, starship isn't present, and the profile.d script means that every time I open a terminal, I get an error message. Granted, this isn't on a fresh install (and on one rebased from Kinoite, back when that was still half-supported), but I imagine quite a few other users will see that same error.

It's hard to say without any other info why you are experiencing this issue. But yeah I agree we probably shouldn't half ass it and just remove it fully.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants