-
Notifications
You must be signed in to change notification settings - Fork 9
Make it composer installable #10
Comments
Feel free to make it composer installable. I've never used composer and don't much work in PHP anymore. |
@krewenki Would you like to transfer ownership of this to the DocRaptor owners for maintenance? Otherwise we will probably fork it and deal with this stuff there. |
I was just planning on "forking", resp. actually just starting a new project. What ETA do you have @janxious ? |
@markushausammann It would probably take me multiple days to get it done since I haven't used composer either. If you're willing to make a PR against a forked project, I am happy to review and merge such a thing at your earliest convenience. |
That's a good solution, I will make a PR against your fork, just tell me when it's ready. Using packagist, everything can be setup in a way that every time you tag a release, packagist automatically publishes the new package version. You might want to open a packagist.org account or I'll register it through mine. Packagist is the default package repository for composer, which is what gem is to ruby, just better ;). |
Actually you should open your own packagist.org account because you need an API key to set the hook up on github. |
Repo is gonna be here: https://github.com/expectedbehavior/php-docraptor |
Would you like to make changes before I make the composer PR? |
@markushausammann Yeah, I am pulling in some PRs from this repo and making a few changes. Almost there. |
@markushausammann I have made all the changes to the EB version of this thing, so PR away! |
@janxious can we chat somewhere, I think I'm ready with the PR and I've got a few more PRs planned, if you're interested. |
And could you open the issue tracker? |
Issue tracker is opened. Let's continue this chat over there. |
Even the most simple dependency deserves to be composer installable!
The text was updated successfully, but these errors were encountered: