great start! can you remove all changes to NeoPixel.cpp and .h? you dont need to include the colors header file there and cpp has some white space changes (which i'd like to not have in this PR!)

First of all, sorry for taking so long to get back to you. Secondly, I removed all changes to the Neopixel.h and .cpp files. However, when you remove the include reference to the hexadecimal_colors_to_strings.h in the .h file, colors.ino does not compile (reflected by the failed build) because TEAL, YELLOW, and MAROON are now not declared. I could put the library include for the hexadecimal_colors_to_strings.h file at the top of the example but that would kind of defeat the purpose of integrating it within the Adafruit Neopixel library. Let me know how you want me to move forward.

I'd like to see these constants namespaced, either by prefixing them withNEO_ or doing something like this:

namespace NeoColors {  const uint32_t BLACK = 0x000000  const uint32_t NAVY = 0x000080}

Because otherwise any user sketches with any of these names won't work anymore. I've seen a couple of LCD graphics libraries that have these same constants, and those sketches won't work anymore without modifying either library (or adding hundreds of#undefs). This also allows sketches to just addusing namespace NeoColors; if they aren't using anything that has a conflict to just use the short name.

I have the#include <stdint.h> at the top of the hexadecimal_colors_to_strings.h file because Arduino was not recognizing uint32_t as a type but the example colors.ino file still won't compile because the hexadecimals.h file isn't included anywhere

please ping when this compiles and passes travis CI

You need to include "hexadecimal_to_string.h" header