Skip to content

docs: update brand theme and tokens#1995

Open
timowolf wants to merge 1 commit intomainfrom
docs/update-brand-theme
Open

docs: update brand theme and tokens#1995
timowolf wants to merge 1 commit intomainfrom
docs/update-brand-theme

Conversation

@timowolf
Copy link
Copy Markdown
Member

@timowolf timowolf commented May 4, 2026

@timowolf timowolf requested a review from a team as a code owner May 4, 2026 07:36
@timowolf timowolf added this to the 49.x milestone May 4, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request updates the @siemens-ux/design-tokens and @simpl/brand dependencies in package.json. Feedback was provided to correct the PR title's commit type from docs to build, as required by the repository style guide for dependency updates.

Comment thread package.json
Comment on lines +183 to +184
"@siemens-ux/design-tokens": "^0.4.1",
"@simpl/brand": "3.2.0",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

medium

The PR title uses the docs commit type, which is reserved for documentation-only changes. Since this PR updates external dependencies, it should use the build type instead, as specified in the repository style guide (line 211). Consider using a scope like theme for better categorization (e.g., build(theme): update brand theme and tokens).

References
  1. Dependency changes must use the 'build' type as per the allowed types in the commit message format. (link)

@timowolf timowolf requested a review from a team as a code owner May 4, 2026 10:15
Copy link
Copy Markdown
Member

@spike-rabbit spike-rabbit left a comment

Choose a reason for hiding this comment

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

Please squash your commits.

@timowolf timowolf force-pushed the docs/update-brand-theme branch from 9fae6a4 to 776bc8b Compare May 4, 2026 12:36
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