Conversation
There was a problem hiding this comment.
Summary of Changes
Hello @tcelestino, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!
This pull request adds dedicated support for setting up a development environment on Linux. It introduces an automated installation script and specific configuration files for Zsh, Homebrew, Git, and NPM, aiming to streamline the process of getting a new Linux machine configured with the desired tools and settings.
Highlights
- Linux Setup Script: Introduced a new comprehensive bash script (
install_linux) to automate the setup of a development environment on Linux, including interactive prompts for each step. - Linux Specific Configurations: Added new configuration files specifically for Linux, including a Homebrew Brewfile, Zsh configuration (
.zshrc), and Zsh aliases (aliases.zsh). - Automated Tool Installation: The new script automates the installation and configuration of key tools like Oh My Zsh, Homebrew, Node.js (via
n), Deno, Git, NPM, and fonts. - Git Alias Update: Updated existing Git aliases in
.gitconfigto usedevelopinstead ofdevfor pull commands. - New Zsh Function: Added a new Zsh function (
open_work_folder) to easily navigate to common project directories.
Using Gemini Code Assist
The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.
Invoking Gemini
You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.
| Feature | Command | Description |
|---|---|---|
| Code Review | /gemini review |
Performs a code review for the current pull request in its current state. |
| Pull Request Summary | /gemini summary |
Provides a summary of the current pull request in its current state. |
| Comment | @gemini-code-assist | Responds in comments when explicitly tagged, both in issue comments and review comments. |
| Help | /gemini help |
Displays a list of available commands. |
Customization
To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.
Limitations & Feedback
Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.
You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.
Footnotes
-
Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configureGemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution. ↩
There was a problem hiding this comment.
Code Review
This pull request introduces setup scripts and configuration files for a Linux environment, including dotfiles for Git, Zsh, and a Brewfile for package management. The review focuses on the correctness and robustness of the new install_linux bash script and platform-specific considerations for the Linux configuration files.
There was a problem hiding this comment.
Code Review
This pull request introduces configurations and an installation script for setting up a Linux development environment. The changes are generally well-organized. Key feedback includes critical fixes for file handling in the Yaak installation process, and security concerns around storing tokens directly in configuration files. Addressing these points will significantly improve the robustness and reliability of the new Linux setup.
There was a problem hiding this comment.
Code Review
This pull request introduces a comprehensive installation script for Linux environments (install_linux), along with new Linux-specific configuration files for Git, Zsh, and Homebrew. The script is generally well-structured. I've identified a few issues in the install_yaak function concerning the download path and temporary file cleanup, which could lead to script failure or unintended system modifications. Additionally, there's a logical issue in install_oh_my_zsh regarding handling pre-existing installations, and suggestions to improve maintainability and consistency across configuration files.
|
@gemini-code-assist review |
There was a problem hiding this comment.
Code Review
This pull request introduces a script to set up a Linux environment with various tools and configurations. The script is well-structured and includes interactive prompts. The changes also update Git configurations and Zsh aliases. The review focuses on improving the script's robustness, maintainability, and adherence to best practices.
There was a problem hiding this comment.
Adjust:
- Moving order process (setup before install)
- Verify
zshand install usingsudo apt install zshbefore to installoh-my-zsh - Translate steps (s/n/q to y/n/q)
- Fix node.js install. Add
sudo apt install makebefore run n-install - Install node.js using
curl -fsSL https://raw.githubusercontent.com/mklement0/n-install/stable/bin/n-install | bash -s 22 - Add on
zshrc:. "/home/tcelestino/.deno/env"andeval "$(/home/linuxbrew/.linuxbrew/bin/brew shellenv)" - Generator Brewfile dump on Linux
- Set execute file
install_linux
Changelog
Settings to Linux OS
Issue: #48
Checklist
Choose the appropriate options: