Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the installation script by moving three install functions (install_neovim, install_git_delta, and install_gh_copilot) from the main install.sh script to the helpers.sh file to improve code organization and modularity.
- Moved three installation functions from install.sh to helpers.sh
- Removed the corresponding function calls from the main installation flow
- Maintained identical function implementations during the move
Reviewed Changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| install.sh | Removed three install functions and their calls from main() |
| helpers.sh | Added the three moved install functions |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| install_gh_copilot | ||
| symlink_dotfiles | ||
| install_fonts | ||
| install_spaceship |
There was a problem hiding this comment.
The functions install_neovim, install_git_delta, and install_gh_copilot were moved to helpers.sh but their calls were removed from main() without being replaced. This breaks the installation flow as these tools will no longer be installed. Either add the function calls back or source helpers.sh and call them from there.
| install_spaceship | |
| install_spaceship | |
| install_neovim | |
| install_git_delta | |
| install_gh_copilot |
No description provided.