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

Add fish shell support #82

Merged
merged 6 commits into from
Feb 9, 2025
Merged

Add fish shell support #82

merged 6 commits into from
Feb 9, 2025

Conversation

lpgn
Copy link
Contributor

@lpgn lpgn commented Feb 8, 2025

feat: Add fish shell support
This commit introduces support for the fish shell.

Changes include:

  • Updated README to reflect fish shell support.
  • Modified bootstrap script to handle fish shell configuration.
  • Updated system prompt to be shell-agnostic.
  • Added fish shell script for keybindings.

Note: There is a known bug where the LLM output is partially doubled, but the functionality works correctly.

lpgn added 3 commits February 7, 2025 23:19
This commit adds a new provider for Google Gemini, allowing users to select Gemini as their LLM provider in ShellOracle.

The following changes were made:

- Created `src/shelloracle/providers/google.py` with the `Google` provider class.
- Added `google-generativeai` as a dependency in `pyproject.toml`.
- Modified `src/shelloracle/providers/__init__.py` to include the new provider.
- Updated `~/.shelloracle/config.toml` to include a configuration section for the Google provider.
This commit introduces support for the fish shell.

Changes include:

- Updated README to reflect fish shell support.
- Modified bootstrap script to handle fish shell configuration.
- Updated system prompt to be shell-agnostic.
- Added fish shell script for keybindings.

Note: There is a known bug where the LLM output is partially doubled, but the functionality works correctly.
@djcopley
Copy link
Owner

djcopley commented Feb 9, 2025

Thanks!

@djcopley djcopley merged commit ec181c5 into djcopley:main Feb 9, 2025
5 of 6 checks passed
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