Conversation
There was a problem hiding this comment.
Pull request overview
This PR adds tmux and VS Code configuration to the dotfiles repository, enhancing the development environment setup with terminal multiplexer and editor preferences.
Key Changes:
- Added tmux configuration with custom prefix (Ctrl+A), plugins via TPM, and a feature-rich status bar
- Added VS Code user settings with Python development defaults, auto-save, and custom editor preferences
- Updated install script to handle tmux and VS Code symlinks, including backup functionality for VS Code settings
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
| vscode/settings.json | New VS Code user settings with editor preferences, Python configuration, and development tools setup |
| tmux/tmux.conf | New tmux configuration with custom keybindings, plugin integration, and system monitoring in status bar |
| install.sh | Added tmux config linking and VS Code settings installation with backup support; renamed function to link_config |
| README.md | Updated documentation to include tmux and VS Code sections; refined feature descriptions and removed completed TODO items |
| .pre-commit-config.yaml | Minor whitespace fix (no functional change) |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| // Default dark theme (lightweight and clean) | ||
| "workbench.colorTheme": "Default Dark Modern", | ||
| // File icons (requires "Material Icon Theme" extension) | ||
| "workbench.iconTheme": "material-icon-theme", |
There was a problem hiding this comment.
[nitpick] The workbench.iconTheme is set to "material-icon-theme" which requires the Material Icon Theme extension to be installed. If the extension is missing, VS Code will show an error. Consider either documenting the required extensions in the README or handling this more gracefully (e.g., omitting this setting or providing fallback).
| "workbench.iconTheme": "material-icon-theme", |
There was a problem hiding this comment.
@copilot please, add this suggestion in the README
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 participant
Added tmux and vscode