Matthew Peveler

Results 298 comments of Matthew Peveler

> > you'd be able to add a string type hint and then if a user passed in an int, it'd do this implicit conversion > > It would convert...

Yes, sorry, I meant to include that using that would require PHP 7.0 (along with the scalar `string` type hint), but it got deleted in one of my rewrites of...

I hit this today and was surprised by the behavior, where we were doing `fetchAsString: [ 'Date' ]`. Our expectation [from the docs](https://docs.snowflake.com/en/developer-guide/node-js/nodejs-driver-consume#fetching-data-types-as-strings) was that only actual date values would...

From the https://readthedocs.org/projects/pypdf/builds/ link, it would seem that RTD was being regularly triggered on any commit to `main` as well as new tags until the 4.0.1 release (https://github.com/py-pdf/pypdf/commit/757932944f54ba661b89e0629ed3fc9d8345dbab), at which...

I agree with @MartinThoma in that for a library, should treat runtime dependencies and dev dependencies separately, where the former should be tested at the lowest + highest supported versions...

The warning is expected if you don't pass `--environment` to the command, to indicate that it's using the value from `default_environment`. The only potential change here would be reducing this...

Yeah, all the commands that take an environment option all have the same output if it's omitted: https://github.com/cakephp/phinx/blob/5c28445848a1daba6883dd94f77c0feef0db3d28/src/Phinx/Console/Command/Migrate.php#L76 Looking at git history, it's also been like this since the project's...

Options that still lack explanation: - Highlight vs Tag (Picture of both to see differences) - Items With Coupons - Items Stored As Gift - Items You Have A Guest...

Oh wow, didn't realize there was an issue for this. Yeah, if we wanted to support 32bit PHP versions, I agree that we'd want to treat the version as a...

Closing this as phinx now requires php64 when installing. If composer does not complain about this when installing phinx as a sub-dependency, I think this should be an issue made...