Skip to content

Conversation

@thewilkybarkid
Copy link
Contributor

Currently autoload-dev blocks aren't merged (whereas require-dev are). This adds them in.

Copy link
Member

Choose a reason for hiding this comment

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

Looks like you are duplicating quite some code from the mergeAutoload function. How about factoring out the common part?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok, I've done that.

@JeroenDeDauw
Copy link
Member

👍

bd808 added a commit that referenced this pull request Aug 12, 2015
@bd808 bd808 merged commit 49ca29a into wikimedia:master Aug 12, 2015
@bd808 bd808 added this to the 1.3.0 milestone Aug 12, 2015
@thewilkybarkid thewilkybarkid deleted the autoload-dev branch August 12, 2015 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

3 participants