Skip to content

TidalShell support, now tested.(extends #2428)#2432

Open
cerealdevv wants to merge 6 commits into
fastfetch-cli:devfrom
cerealdevv:dev
Open

TidalShell support, now tested.(extends #2428)#2432
cerealdevv wants to merge 6 commits into
fastfetch-cli:devfrom
cerealdevv:dev

Conversation

@cerealdevv

@cerealdevv cerealdevv commented Jul 1, 2026

Copy link
Copy Markdown

Summary

This PR adds native version detection and proper process tree handling for TidalShell (https://github.com/c-cluster/TidalShell).
This PR is a repost of the pull #2428
it has now been tested locally on Windows x64
Screenshot 2026-07-01 125528

Changes

  • terminalshell.c: Added getShellVersionTidalshell helper. It invokes tidalshell -v (introduced in TidalShell 0.2-Nightly), trims trailing spaces, and isolates the numeric version by stripping out build tags (e.g., -Nightly).
  • terminalshell_linux.c: Added tidalshell to the known shells ignore array inside getTerminalInfo. This prevents Fastfetch from misidentifying the shell as a terminal emulator on Linux systems.
  • terminalshell_windows.c: Added proper case mapping in setShellInfoDetails so the UI cleanly displays TidalShell.

Checklist

  • I have tested my changes locally.

Notes for Reviewers

Please review my code. as I stated, I am not primarily a C/C++ developer. I am not as experienced in it.
I do not know if my code is efficient nor if it meets fastfetch quality standards.

@codacy-production

Copy link
Copy Markdown
Contributor

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@cerealdevv

Copy link
Copy Markdown
Author

if there IS still issues then please let me know.
if this PR gets closed there will be no more attempts from me to get TSh proper version support on fastfetch, however you are still allowed to merge my code into fastfetch.

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.

1 participant