ytetsuro comments

Results 24 comments of


                                            ytetsuro

Fixed a bug that firestore does not save as a date if the date exists in the map in the array.

@prescottprue There should be no difference in coverage, but for some reason it has been determined that coverage is down. Perhaps the coverage measurement during typescript transpile is incorrect, but...

[QueryBuilder] The combination of the `setUpdateBatch()` and the `updateBatch()` is difficult to use.

Thanks, response. You are right.

[QueryBuilder] The combination of the `setUpdateBatch()` and the `updateBatch()` is difficult to use.

@kenjis I was wondering if the following code could be implemented without adding breaking change if the user has called setUpdateBatch at least once. The concrete method is to change...

[QueryBuilder] The combination of the `setUpdateBatch()` and the `updateBatch()` is difficult to use.

@kenjis Thanks, response. > If you change it to accept int, it is breaking change. Sure! I'm sorry.😞 > You could write code like this on PHP8: Oh! You are...

_processForeignKeys | Identifier name is too long

Oracle 12.1 is only 30 characters. Therefore, if you want to put in a limit of 64 characters or not, I think it is preferable to do it per driver.

To what extent is CodeIgniter\Database\Forge::_attributeType intended to absorb differences?

@kenjis Response very thanks. I figured out that it is based on MySQL.

To what extent is CodeIgniter\Database\Forge::_attributeType intended to absorb differences?

@paulbalandan Why close? Are you saying that it is wrong to ask this question in an issue? To what extent is CodeIgniter\Database\Forge::_attributeType intended to absorb differences? Why does the following...

To what extent is CodeIgniter\Database\Forge::_attributeType intended to absorb differences?

If I modify this as follows, does CodeIgniter need it? - Align it to ANSI SQL or SQL standard. - Allow each driver to absorb as much of the ANSI...

Add PHP 8 Support / WIP

@czosel @cseufert @MaartenStaa Fixed #968. Are you ready to release 3.1.0 already?

What should I do if I want to multiInject multiple targetName?

@dcavanagh Thanks for taking the time to reply. I think the only current solution is probably to create a factory. Does inversifyJS want to change to allow multiple target names...