Either allow literal ints as shape keys, or require class constant shape keys to be strings

``` Hack
class MyClass {
const int FOO = 123;
}

type MyShape = shape(
MyClass::FOO => string, // this is fine
123 => string, // expected string literal or class constant
}
```

Discovered while looking into why Shapes::toArray() returns `array` instead of `array`