Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Configuration variables must be set for non-interactive sessions #521

Open
Anfid opened this issue Aug 21, 2024 · 0 comments
Open

Configuration variables must be set for non-interactive sessions #521

Anfid opened this issue Aug 21, 2024 · 0 comments
Labels
🐛 bug Something isn't working

Comments

@Anfid
Copy link

Anfid commented Aug 21, 2024

Describe the bug

If a configuration variable is defined exclusively for interactive sessions, it gets ignored.

This seems to me more like a documentation bug, rather than logic bug. I assume tide spawns non-interactive instances of fish to render the prompt, which can't see any configuration variables defined inside status is-interactive blocks. Since this isn't mentioned in README or wiki, default assumption could be that prompt is only necessary for interactive sessions, so configuration variables should only be defined for them.

Steps to reproduce

  1. Install tide and try to configure it, following the wiki;
  2. Add configuration variables inside the default if status is-interactive block;
  3. Reload prompt. Environment variables can be accessed, but configuration isn't applied.

Environment

Output of tide bug-report:

fish version: 3.7.1
tide version: 6.1.1
term: xterm-256color
os: macOS Sonoma 14.6.1
terminal emulator: iTerm2
fish startup: 56.60 millis
fisher plugins: jorgebucaran/fisher gazorby/fish-abbreviation-tips patrickf1/fzf.fish blt-r/hydro jorgebucaran/autopair.fish ilancosman/tide@v6 paldepind/projectdo nickeb96/puffer-fish

Potential solution

Updating README and wiki to state that configuration variables must be defined for non-interactive sessions too.

@Anfid Anfid added the 🐛 bug Something isn't working label Aug 21, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant