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

Moving rgb typedefs into a single location #5978

Merged
merged 1 commit into from
Jun 4, 2019

Conversation

XScorpion2
Copy link
Contributor

@XScorpion2 XScorpion2 commented May 25, 2019

Description

This PR consolidates RGB struct typedefs into a single location allowing us to easily change out the struct type based on the rgb driver needs.

Types of Changes

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

Issues Fixed or Closed by This PR

Checklist

  • My code follows the code style of this project.
  • My change requires a change to the documentation.
  • I have updated the documentation accordingly.
  • I have read the CONTRIBUTING document.
  • I have added tests to cover my changes.
  • I have tested the changes and verified that they work and don't break anything (as well as I can manage).

Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
@XScorpion2 XScorpion2 force-pushed the features/rgb_type_cleanup branch from 188cebd to e7e4136 Compare June 3, 2019 00:22
Copy link
Contributor

@mechmerlin mechmerlin left a comment

Choose a reason for hiding this comment

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

Thanks!

@mechmerlin mechmerlin merged commit e7711b3 into qmk:master Jun 4, 2019
@XScorpion2 XScorpion2 deleted the features/rgb_type_cleanup branch June 4, 2019 21:55
drashna pushed a commit to zsa/qmk_firmware that referenced this pull request Jun 5, 2019
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
fdidron pushed a commit to zsa/qmk_firmware that referenced this pull request Jun 6, 2019
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
Timbus pushed a commit to Timbus/qmk_firmware that referenced this pull request Jun 23, 2019
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
ridingqwerty pushed a commit to ridingqwerty/qmk_firmware that referenced this pull request Jan 10, 2020
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
JeffreyPalmer pushed a commit to JeffreyPalmer/qmk_firmware that referenced this pull request Feb 27, 2020
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
swamp09 pushed a commit to swamp09/qmk_firmware that referenced this pull request Mar 11, 2020
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
BorisTestov pushed a commit to BorisTestov/qmk_firmware that referenced this pull request May 23, 2024
Because someone named the define poorly

Using full relative path as handwired/promethium has a color.h file
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.

4 participants