Welcome Guest, Not a member yet? Register   Sign In
CodeIgniter 4.0 Query Builder and CAST support
#7

(03-16-2020, 11:48 AM)zahhar Wrote: Hey man, nice try and thank you for taking care!

I do not want to be a party-braker here, but:
1) you did PR towards wrong repo; right one is https://github.com/codeigniter4/CodeIgniter4
2) you did not cover your change with Unit Tests, so your PR will not be accepted until you extend existing tests for like() method with additional test-cases that cover your changes
3) you have to take care about documentation as well
4) your code smells - but you will know it from maintainers later; particularly - you allow any string as $cast attribute, however, mySQL supports only handful of keywords in their CAST() function - and you mentioned them (INTEGER, DATE, DOUBLE, DOUBLE PRECISION, BOOLEAN, TEXT), but did not add checking and error reporting
5) you should validate your Github account and sign your commit. Read more how to do it here: https://help.github.com/en/github/authen...ng-commits

Good luck with 2nd attempt Smile

Hey buddy, I had no doubts that I was going to miss something.  Big Grin Big Grin

Thanks for all the information: let's see if I can perfect my attempt.
Codeigniter 4 - Docker Image [github] [docker hub]
Reply


Messages In This Thread
RE: CodeIgniter 4.0 Query Builder and CAST support - by atsanna - 03-16-2020, 12:48 PM



Theme © iAndrew 2016 - Forum software by © MyBB