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

Upgrading doctrine/common 2.4 (via composer) #281

Merged
merged 1 commit into from Mar 4, 2013

Conversation

Ocramius
Copy link
Member

@Ocramius Ocramius commented Mar 4, 2013

This PR introduces changes necessary to make the DBAL recognize "doctrine/common": "2.4" as compatible as discussed in https://groups.google.com/d/topic/doctrine-dev/8WjzboENQgg/discussion

Ping @dbu

@doctrinebot
Copy link

Hello,

thank you for positing this Pull Request. I have automatically opened an issue on our Jira Bug Tracker for you with the details of this Pull-Request. See the Link:

http://doctrine-project.org/jira/browse/DBAL-458

@doctrinebot
Copy link

Oh btw, I just (automatically) realized that you are not creating this pull request against the master branch.

Unless there are good reasons for this, i would suggest to close and rebase the Pull Request against master and then create it again. Sorry!

beberlei added a commit that referenced this pull request Mar 4, 2013
Upgrading doctrine/common 2.4 (via composer)
@beberlei beberlei merged commit 569f1ae into doctrine:2.3 Mar 4, 2013
@dbu
Copy link
Member

dbu commented Mar 5, 2013

should we not add explicit testing with commons 2.3 and 2.4 to .travis, to be sure nothing breaks? the drawback is that we would duplicate the number of builds to run.

@Ocramius
Copy link
Member Author

Ocramius commented Mar 5, 2013

@dbu 2.3 is freezed :)

@Ocramius Ocramius deleted the doctrine-common-2.4 branch March 5, 2013 08:24
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Aug 27, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants