Skip to content
This repository was archived by the owner on Jan 29, 2020. It is now read-only.

remove @deprecated class and method #2

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

samsonasik
Copy link
Contributor

No description provided.

@samsonasik samsonasik force-pushed the deprecated branch 3 times, most recently from 18b2456 to 76a7ee7 Compare June 3, 2015 21:35
@vaclavvanik
Copy link

bump php version in composer :-)

@Maks3w
Copy link
Member

Maks3w commented Jun 4, 2015

@vaclavvanik This PR does not imply a change in PHP requirements.

@vaclavvanik
Copy link

Oh, you are right. I thought rashly

@Ocramius Ocramius added this to the 3.0.0 milestone Jun 4, 2015
@samsonasik
Copy link
Contributor Author

rebased

@Maks3w
Copy link
Member

Maks3w commented Jun 7, 2015

I'll close this until v3 development of this component begin

@Maks3w Maks3w closed this Jun 7, 2015
@samsonasik
Copy link
Contributor Author

:(. this is why I think we need to have per-branch : 2.4, 2.5, 2.6, 3.0 , early works merged is better imo.
//cc @gianarb @weierophinney

@weierophinney weierophinney reopened this Jun 8, 2015
@weierophinney
Copy link
Member

This can begin now on the develop branch. Essentially, this PR is indicating that we have developers wanting to work on BC-breaking changes, which means that the develop branch should become the 3.0 branch. I'll review, and, on merge, bump the branch aliases for the develop branch.

@samsonasik
Copy link
Contributor Author

mergeable? Or i should re-create PR against develop branch?

@samsonasik samsonasik changed the base branch from master to develop September 9, 2016 11:45
@samsonasik
Copy link
Contributor Author

@weierophinney I've changed target branch to develop

@samsonasik
Copy link
Contributor Author

@weierophinney merge-able ?

@samsonasik
Copy link
Contributor Author

Is there anything I can do to get it merged? Thank you.

@weierophinney
Copy link
Member

This repository has been closed and moved to laminas/laminas-authentication; a new issue has been opened at laminas/laminas-authentication#9.

@weierophinney
Copy link
Member

This repository has been moved to laminas/laminas-authentication. If you feel that this patch is still relevant, please re-open against that repository, and reference this issue. To re-open, we suggest the following workflow:

  • Squash all commits in your branch (git rebase -i origin/{branch})
  • Make a note of all changed files (`git diff --name-only origin/{branch}...HEAD
  • Run the laminas/laminas-migration tool on the code.
  • Clone laminas/laminas-authentication to another directory.
  • Copy the files from the second bullet point to the clone of laminas/laminas-authentication.
  • In your clone of laminas/laminas-authentication, commit the files, push to your fork, and open the new PR.
    We will be providing tooling via laminas/laminas-migration soon to help automate the process.

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

Successfully merging this pull request may close these issues.

5 participants