Skip to content

Adding OhMyZsh-Theme Option to common-utils feature#1564

Open
the78mole wants to merge 5 commits intodevcontainers:mainfrom
the78mole:main
Open

Adding OhMyZsh-Theme Option to common-utils feature#1564
the78mole wants to merge 5 commits intodevcontainers:mainfrom
the78mole:main

Conversation

@the78mole
Copy link

This pull request adds support for customizing the Oh My Zsh theme during installation in the common-utils feature. The main changes include introducing a new ohMyZshTheme option, updating scripts to handle this option safely, and adding tests to verify theme customization.

Feature: Oh My Zsh Theme Customization

  • Added a new ohMyZshTheme option to devcontainer-feature.json, allowing users to specify the Oh My Zsh theme (default: devcontainers).
  • Updated README.md to document the new ohMyZshTheme option and its usage.

Implementation: Script Updates

  • Modified install.sh and main.sh to accept and use the OH_MY_ZSH_THEME environment variable, defaulting to devcontainers if not specified. [1] [2]
  • Enhanced main.sh to validate the theme name and update .zshrc accordingly, falling back to the default if the provided name is invalid.

Testing: Custom Theme Scenario

  • Added a new test script custom-zsh-theme.sh to verify that the specified theme is applied and Oh My Zsh is installed.
  • Updated scenarios.json to include a scenario for testing custom theme installation with the fino theme.

Copilot AI and others added 5 commits February 8, 2026 18:10
Co-authored-by: the78mole <7723396+the78mole@users.noreply.github.com>
Co-authored-by: the78mole <7723396+the78mole@users.noreply.github.com>
Co-authored-by: the78mole <7723396+the78mole@users.noreply.github.com>
Add ohMyZshTheme configuration option to common-utils
@the78mole the78mole requested a review from a team as a code owner February 8, 2026 18:28
@the78mole
Copy link
Author

@microsoft-github-policy-service agree

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