Skip to content

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

Copy link
Contributor

@Kaniska244 Kaniska244 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @the78mole

Thank you for the contribution. Would you kindly do a minor common-utils feature version bump for this.

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.

3 participants