Skip to content
This repository was archived by the owner on Sep 24, 2018. It is now read-only.

Composer - #310

Closed
aaronbrazell wants to merge 5 commits into
WP-API:masterfrom
aaronbrazell:composer
Closed

aaronbrazell wants to merge 5 commits into
WP-API:masterfrom
aaronbrazell:composer

Conversation

@aaronbrazell

Copy link
Copy Markdown

Adds a composer.json file for composer-based WordPress installs.

Comment thread composer.json

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I feel like this one could probably be a bit more descriptive. :)

@tlovett1

tlovett1 commented Oct 9, 2014

Copy link
Copy Markdown
Member

+1 on this

@austinpray

Copy link
Copy Markdown

Please

👍

@retlehs

retlehs commented Dec 10, 2014

Copy link
Copy Markdown

could use a squash (why did you add and remove composer multiple times?) and better formatting in composer.json (mixed tabs and spaces, missing spaces in certain spots)

@rmccue

rmccue commented Dec 10, 2014

Copy link
Copy Markdown
Member

I'm happy to merge (although I don't use Composer myself, so y'all will need to let me know if it breaks), but as noted, needs to be squashed and remaining issues fixed.

@chrisvanpatten

Copy link
Copy Markdown

I'd be happy to submit a new PR for this but I don't want to step on any toes. @technosailor - would you mind if I worked up a new version that accounts for the suggestions/issues raised here?

@retlehs

retlehs commented Apr 29, 2015

Copy link
Copy Markdown

👀

@retlehs retlehs mentioned this pull request Apr 29, 2015
@danielbachhuber

Copy link
Copy Markdown
Member

Closing in favor of #1157

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants