New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Provide dokuwiki as composer package #1757
Comments
Could you explain the advantage? |
see this https://getcomposer.org/doc/03-cli.md#create-project You could make dokuwiki a composer package so that everyone could install it with one simple command. |
My wiki setup would be easily manageable via git and deployable via a CI process. |
@smichaelsen huh? You already can use git. I still don't get the advantage composer would get you. Composer is useful where you manage dependencies. But (ab)using it as an installer seems weird. |
I wouldn't say it's weird. See @gmponos example. Composer is well suited for deploying applications. At least it's what we do in the TYPO3 world since a while: Put a composer.json into a repository, add some configuration => get an installable web application |
shrug I guess I would accept a pull request |
@splitbrain It just needs a submission to https://packagist.org/. |
I am also interested in being able to install Dokuwiki through composer. My ideal goal would be to manage my Dokuwiki as a regular composer project. My repo would only contain my custom plugins and any third-party plugin or the core engine would be loaded by composer (and not commited to the repo). This would enable me to also use composer to add regular dependencies to my plugins: it is currently not clear how to consume code from Related issue: #1840 |
This also adds full test coverage and injects the logger as a PSR-3 interface. Unfortunately, we have to commit the entire vendor directory as DokuWiki currently does not allow installing plugins via composer (yet). Maybe it will come day via implementing dokuwiki/dokuwiki#1757
dokuwiki already has a composer.json but is not installable via composer. That should be possible.
The text was updated successfully, but these errors were encountered: