Skip to content
This repository was archived by the owner on Apr 12, 2025. It is now read-only.

Composer for beautify #2

Open
cepa wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Composer for beautify #2

cepa wants to merge 8 commits into from

Conversation

@cepa
Copy link

@cepa cepa commented Jun 23, 2013

Hi,

I really like the way Beautify works, so I wanted to add it as a filter to my project, but unfortunately you ship it with dependencies. So I forked your repo and added composer do download dependencies and to make it possible to mix beautify with other projects.

Let me know if you would like to merge changes.

@edwinm
Copy link
Owner

edwinm commented Jun 23, 2013

Lukasz,

I really like the way Beautify works, so I wanted to add it as a
filter to my project, but unfortunately you ship it with dependencies.
So I forked your repo and added composer do download dependencies and
to make it possible to mix beautify with other projects.

Thanks for your pull request.

I haven't had time to look into this, I'll try the next couple of days.

I'll let you know.

Thanks,
Edwin Martin

You can merge this Pull Request by running
git pull https://github.com/cepa/Beautify master

Or view, comment on, or merge it at:

#2

    Commit Summary

Read my webdevelopment weblog http://www.bitstorm.org/ or follow me on twitter https://twitter.com/edwinmdev

pawel.lenart and others added 2 commits January 30, 2014 11:18
@Fale
Copy link

Fale commented Feb 23, 2014

Please, do merge this and create a packagist package.

This pull request makes the code much smaller and much more usable

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

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@cepa @edwinm @Fale