Zeroclickinfo-goodies: PHP Cheat Sheet: Need improvement
There is lot of functions in this cheat sheet which nobody uses in day to day life. I guess cheat sheet are meant to be useful for quick summarizing the concept. There is no MySQL functions which is used in daily basis. And why we need Regular expression in this cheat sheet.
Need improvement
IA Page: http://duck.co/ia/view/php_cheat_sheet
Maintainer: @rathoreanirudh
shahbaz17
All 9 comments
Thanks for the suggestion. @shahbaz17 are there other kinds of PHP functions besides MYSQL that you use regularly?
alohaas
on 7 Jun 2016
Also, I feel that many functions under sections PHP array functions and string functions aren't in proper order. That means some of the similar functions should be put together to make it a good reference for users. I'll be submitting a PR soon.
sahildua2305
on 9 Jun 2016
Also,regex cheat sheet is already there so we can remove the regex part from the php cheat sheet.
shubhamrajput
on 10 Jun 2016
Point! @shubhamrajput
sahildua2305
on 10 Jun 2016
Hello @alohaas , @sahildua2305 has taken this opportunity to make this cheat-sheet more useful. His latest commit has all what is needed in regular day to day php functions. Also there is always a room for improvement. And Thanks @himanshu53 , @sahildua2305 and @shubhamrajput for your quick reaction.
shahbaz17
on 12 Jun 2016
@shahbaz17 My pull request isn't yet merged though.
sahildua2305
on 12 Jun 2016
Hello @sahildua2305 , I have seen the PR and gone through the conversation. Maintainer input is required and we follow a set of rules . Just be patient , you have done a great job. Your PR will get merged once maintainer reply or after 14 days. Keep contributing :+1:
shahbaz17
on 12 Jun 2016
Thank you :smile:
sahildua2305
on 12 Jun 2016
Thank you
On 13-Jun-2016 12:11 AM, "Sahil Dua" [email protected] wrote:
Thank you 😄
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/duckduckgo/zeroclickinfo-goodies/issues/3183#issuecomment-225453146,
or mute the thread
https://github.com/notifications/unsubscribe/AQSAdkYipDyh_8C70gCbDv9Xf-TKOsK6ks5qLFLrgaJpZM4IuEIA
.
himanshu53
on 13 Jun 2016
Related issues
spagy
·
19Comments
moollaza
·
42Comments
atomic1fire
·
15Comments
moollaza
·
14Comments
rushsteve1
·
13Comments
Most helpful comment
Hello @sahildua2305 , I have seen the PR and gone through the conversation. Maintainer input is required and we follow a set of rules . Just be patient , you have done a great job. Your PR will get merged once maintainer reply or after 14 days. Keep contributing :+1: