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

adjust puli dependencies #40

Merged
merged 1 commit into from
Jan 4, 2016
Merged

adjust puli dependencies #40

merged 1 commit into from
Jan 4, 2016

Conversation

dbu
Copy link
Contributor

@dbu dbu commented Jan 4, 2016

No description provided.

@@ -13,8 +13,8 @@
"require": {
"php": "^5.4|7.*",
"puli/composer-plugin": "~1.0.0-beta8",
"puli/discovery": "~1.0.0-beta8",
"puli/cli": "~1.0.0-beta9"
"puli/discovery": "1.0.0-beta8",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

master is currently broken. we should be more flexible again when its fixed, as this will be a huge PITA to install if more than one repository depends on puli.

@sagikazarmark
Copy link
Member

👍 Will tag another version once this gets merged.

@dbu
Copy link
Contributor Author

dbu commented Jan 4, 2016

i think we should merge this. for 0.6 we depend on puli/cli, stopping to do that we need to bump to 0.7 - if we want that. we discuss this in #41 and will change again. right now, this PR keeps the same functionality with the same drawbacks but actually working (i hope)

@sagikazarmark
Copy link
Member

👍

dbu added a commit that referenced this pull request Jan 4, 2016
@dbu dbu merged commit fff8765 into master Jan 4, 2016
@dbu dbu deleted the dbu-patch-1 branch January 4, 2016 16:07
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.

None yet

2 participants