Skip to content

(#1165) Upgrade to choco-theme v1 #1166

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Apr 16, 2025

Conversation

st3phhays
Copy link
Member

Description Of Changes

The changes here allow compatibility with choco-theme v1. While other improvements could be made, these are the required changes.

See choco-theme release notes for more information: https://github.com/chocolatey/choco-theme/releases/tag/1.0.0

Motivation and Context

We need choco-theme on the latest version.

Testing

Follow instructions at chocolatey/choco-theme#434

Change Types Made

  • Minor documentation fix (typos etc.).
  • Major documentation change (refactoring, reformatting or adding documentation to existing page).
  • New documentation page added.
  • The change I have made should have a video added, and I have raised an issue for this.

Change Checklist

  • Requires a change to menu structure (top or left-hand side)/
  • Menu structure has been updated

Related Issue

#1165

@st3phhays st3phhays requested a review from alexaveldanez April 15, 2025 10:32
@st3phhays st3phhays self-assigned this Apr 15, 2025
The changes here allow compatibility with choco-theme v1. While other
improvements could be made, these are the required changes.

See choco-theme release notes for more information:
https://github.com/chocolatey/choco-theme/releases/tag/1.0.0
@st3phhays st3phhays marked this pull request as ready for review April 16, 2025 10:35
Copy link
Contributor

@alexaveldanez alexaveldanez left a comment

Choose a reason for hiding this comment

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

Looks good to me!

@alexaveldanez alexaveldanez merged commit da54ea8 into chocolatey:master Apr 16, 2025
1 check passed
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