getkirby / cli

Kirby Command Line Interface
MIT License
51 stars 5 forks source link

Switch toolkit methods to php functions #66

Closed afbora closed 6 months ago

afbora commented 6 months ago

I'll also take care linux commands in separate PR.

afbora commented 6 months ago

I understand. So we can just close the PR? @bastianallgeier

lukasbestle commented 6 months ago

@bastianallgeier But wouldn't it still be safer to be independent from the installed Kirby version?

bastianallgeier commented 6 months ago

@lukasbestle In some cases probably yes. But I'm not sure what would be the best way to solve this. Is there a good way to also include Kirby as a composer dependency and use that one whenever possible and only use the installed version when it is really needed to do so?

lukasbestle commented 6 months ago

Not really. I think the classes will compete with each other as each class can only be loaded once.