Hoa\Core should NEVER edit php configuration.
stephpy opened this issue · comments
We should avoid automatic definition of what we estimate as a good thing like this
It should be a command which pilot user to edit its configuration to use good thinkgs (UTC as timezone, etc....)
This command could be part of Hoa\Core ?
Thought ?
What about including this command in the devtools? hoaproject/Central#15 If yes, we have to release the devtools as soon as possible in order to add this tool.
This is a devtools, looks the good place yes. :)
So, comment hoaproject/Central#15 if you think this library will be useful.
I don't know if it's a good idea tu put it in Devtools or in another case
if Devtools is a good name.
Let me explains, from my point of view Devtools is for Hoa community, and
it's not for end user. If I want to use Hoa\Compiler (if I remember well
need to have mb_string correctly defined), I need also Devtools to check my
configuration. But it will only use once and it's the same problem than
@steph_py explain regarding core dependency. For me, we have two kinds of
tools, tools for helping user and tools for dev. All hoackers could have
this two kinds of tools. But devtools is hoackers specific tools and call
it "Tools" is for users.
Le mar. 18 nov. 2014 à 17:15, Ivan Enderlin notifications@github.com a
écrit :
So, comment hoaproject/Central#15
hoaproject/Central#15 if you think this
library will be useful.—
Reply to this email directly or view it on GitHub
#61 (comment).
I had the same questions here. What about adding it to Hoa\Core? Because it would help to check all libraries.
Solved by the core split isn't it ?
Yup!
Good catch 👍
So far, the prelude does not change mb_* configuration: https://github.com/hoaproject/Consistency/blob/7656b971a8248a8d314ed0c5bb0668936438f62a/Prelude.php.
I checked it before posting ^^
Just for the record and for @stephpy :-).
👍