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.

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

Composer support #1

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

Composer support #1

wants to merge 3 commits into from

Conversation

fran6co
Copy link

@fran6co fran6co commented Oct 6, 2013

No description provided.

@djekl
Copy link

djekl commented Oct 6, 2013

Shouldn't the homepage be set to the url of this repo, and docs/wiki also migrated over. Since the Google code site now says that Github is its new codebase…

fran6co and others added 3 commits October 6, 2013 10:45
Example: .../srcUnitedPrototype/GoogleAnalytics/...
@fran6co
Copy link
Author

fran6co commented Oct 6, 2013

@djekl, done!

@thomasbachem
Copy link
Owner

I have to say that I'm not a fan of the "UnitedPrototype" folder, but I guess that's the only way to make it PSR-0 compliant?

@djekl
Copy link

djekl commented Oct 6, 2013

Unfortunately yes, thats because you have that in its namespace. The namespace needs to match the folder structure.

Sent from my iPhone

On 6 Oct 2013, at 02:57 pm, thomasbachem [email protected] wrote:

I have to say that I'm not a fan of the "UnitedPrototype" folder, but I guess that's the only way to make it PSR-0 compliant?


Reply to this email directly or view it on GitHub.

@robertfausk
Copy link

+1

@vvh-empora
Copy link

why isnt this lib available via composer? @fran6co has already implement the missing composer.yaml?

@djekl
Copy link

djekl commented Oct 28, 2013

Its composer.json and there its not there in the root of the repo. It also needs to be sumitted to https://packagist.org/

@fran6co
Copy link
Author

fran6co commented Oct 28, 2013

It is already in packagist, just waiting for @thomasbachem to merge this PR to master to make it work.

@zomberg
Copy link

zomberg commented Nov 27, 2013

+1 for composer and packagist.org.

@moriony
Copy link

moriony commented Jan 20, 2014

Yes, it's will be great!

@peter-gribanov
Copy link

+1

3 similar comments
@hason
Copy link

hason commented Feb 3, 2014

+1

@faizshukri
Copy link

+1

@lbarulski
Copy link

👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.