David E. Wheeler

Results 436 comments of David E. Wheeler

Oh, okay, I assumed it was just deprecated and no longer kept current. FWIW I noticed this when told me the latest TermReadKey was [2.14](http://api.metacpan.org/v0/release/TermReadKey) instead of [2.37](https://fastapi.metacpan.org/v1/release/TermReadKey).

> It adds two different things (both are fine, but reviewing them independently could be easier). Yeah, I started to add support for `additionalProperties` and quickly realized it needed some...

> I don't think that this is necessary. Part of this comment may have been due to the fact that I'm just getting started with wetzel, and am not always...

I assume it needs to handle arrays in `getSchemaMarkdown()`: ``` patch --- a/lib/generateMarkdown.js +++ b/lib/generateMarkdown.js @@ -221,6 +221,8 @@ function getSchemaMarkdown(schema, fileName, headerLevel, suppressWarnings, sche var title = defaultValue(schema.title, suppressWarnings...

Yeah. I hacked this into my current copy so I could at least get a couple links, but it's copied and modified from elsewhere. Not pretty. ``` patch diff --git...

@meganrm unless and until the [glTF](https://github.com/KhronosGroup/glTF) schema uses an array types, I think it will only be added if someone else contributes it. The project seems pretty amenable to merging...

I fixed my issue by editing `/etc/apache2/mime.types` on my Mac, but it would be nice to be able to specify a file that one can use to add additional type...

Great find, thank you! Slashes are allowed in change names so they can be organized in directories; I guess I forgot about the implications for tags when I designed them....

Sqitch always passes the `--option variable_substitution=true` to enable [SnowSQL variables](https://docs.snowflake.com/en/user-guide/snowsql-use.html#using-variables), which Sqitch uses itself, and you can use in your change scripts. The docs say: > To use an ampersand...