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 the RGB_MATRIX_RAINBOW_PINWHEELS animation #7365

Merged

Conversation

rpbaptist
Copy link
Contributor

Description

The internal naming was PINWHEELS instead of RAINBOW_PINWHEELS causing a mismatch between animation definition and the function.

Types of Changes

  • Core
  • Bugfix
  • New feature
  • Enhancement/optimization
  • Keyboard (addition or update)
  • Keymap/layout/userspace (addition or update)
  • Documentation

The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
@rpbaptist rpbaptist force-pushed the fix-animation-RGB_MATRIX_RAINBOW_PINWHEELS branch from e29f9df to ba87e3b Compare November 15, 2019 11:12
@drashna drashna requested a review from a team November 15, 2019 18:08
Copy link
Contributor

@yanfali yanfali left a comment

Choose a reason for hiding this comment

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

LGTM

@drashna drashna merged commit 381b73f into qmk:master Nov 15, 2019
@XScorpion2
Copy link
Contributor

err, my bad, sorry

drashna pushed a commit to zsa/qmk_firmware that referenced this pull request Nov 18, 2019
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
ripxorip pushed a commit to ripxorip/qmk_firmware that referenced this pull request Dec 3, 2019
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
patrl pushed a commit to patrl/qmk_firmware that referenced this pull request Dec 29, 2019
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
fdidron pushed a commit to zsa/qmk_firmware that referenced this pull request Jan 6, 2020
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
fdidron pushed a commit to zsa/qmk_firmware that referenced this pull request Jan 8, 2020
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
HokieGeek pushed a commit to HokieGeek/qmk_firmware that referenced this pull request Feb 21, 2020
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
The internal naming was `PINWHEELS` instead of `RAINBOW_PINWHEELS`
causing a mismatch between animation definition and the function.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants