Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Composer for beautify #2

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

Composer for beautify #2

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 join this conversation on GitHub. Already have an account? Sign in to comment
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