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

Fix issues with starting sponge #5

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

halostatue
Copy link

I got the following error block after installing 1.1.0 on every new tab.

contains: Key not specified
test: Unexpected argument type at index 1
= true -a true = true
 ^
~/.config/fish/functions/sponge_filter_failed.fish (line 4):
  if test $previously_in_history = true -a $sponge_allow_previously_successful = true
     ^
in function 'sponge_filter_failed' with arguments '0 ""'
	called on line 14 of file ~/.config/fish/functions/_sponge_on_postexec.fish
in function '_sponge_on_postexec'
	called on line 1 of file ~/.config/fish/config.fish
in event handler: handler for generic event “fish_postexec”
	called on line 378 of file ~/.config/fish/config.fish

This fixes both issues. Everything else appears to work. This may
address #4.

I got the following error block after installing 1.1.0 on every new tab.

```
contains: Key not specified
test: Unexpected argument type at index 1
= true -a true = true
 ^
~/.config/fish/functions/sponge_filter_failed.fish (line 4):
  if test $previously_in_history = true -a $sponge_allow_previously_successful = true
     ^
in function 'sponge_filter_failed' with arguments '0 ""'
	called on line 14 of file ~/.config/fish/functions/_sponge_on_postexec.fish
in function '_sponge_on_postexec'
	called on line 1 of file ~/.config/fish/config.fish
in event handler: handler for generic event “fish_postexec”
	called on line 378 of file ~/.config/fish/config.fish
```

This fixes both issues. Everything else *appears* to work. This may
address meaningful-ooo#4.
@halostatue
Copy link
Author

I'm not convinced that this is related to #4 anymore, but I have emit fish_postexec at the end of my config.fish so that direnv hooks are called on shell start. These changes are required in this case.

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

Successfully merging this pull request may close these issues.

None yet

1 participant