Skip to content
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

fix: Revert color palette to immutable state #2578

Merged
merged 2 commits into from Dec 6, 2023

Conversation

abdulbasithqb
Copy link
Contributor

Please verify the following:

  • yarn test jest tests pass with new tests, if relevant

Describe your PR

fixes #2575

This PR refines color palette management, initially mutable (const palette: Record<string, string>), then reverted to immutable state (const palette as const) for consistency.

@jamonholmgren
Copy link
Member

I'm fine with this. Any downsides? @frankcalise or @mazenchami ?

@frankcalise frankcalise changed the title enhancement: Revert color palette to mutable state fix: Revert color palette to immutable state Dec 6, 2023
@frankcalise frankcalise merged commit 00209bb into infinitered:master Dec 6, 2023
1 check passed
infinitered-circleci pushed a commit that referenced this pull request Dec 6, 2023
# [9.1.0](v9.0.5...v9.1.0) (2023-12-06)

### Bug Fixes

* Revert color palette to immutable state ([#2578](#2578) by [@abdulbasithqb](https://github.com/abdulbasithqb)) ([00209bb](00209bb))

### Features

* **cli:** remove unused deps upon demo removal ([#2572](#2572) by [@frankcalise](https://github.com/frankcalise)) ([4d45ae0](4d45ae0))
@infinitered-circleci
Copy link

🎉 This PR is included in version 9.1.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Type enhancement for color palette
4 participants