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

Feature: BeforeScopeInterface (2.next) #264

Merged

Conversation

jamisonbryant
Copy link
Contributor

Redo of #261 but with changes rebased on 2.next

@jamisonbryant
Copy link
Contributor Author

OK, this time I actually did it right and this branch is now based on 2.next with only my commits.

It does still have the stan failure for Identity.php but there is another PR open to correct that.

@LordSimal
Copy link
Contributor

The stan issue will be fixed by #263 so we can ignore that.

@LordSimal
Copy link
Contributor

@jamisonbryant all you have to do now is click on the triangle next to the Update branch button inside github, then select Update with rebase and click on Rebase branch

Then the CI in Github should be green.

@LordSimal
Copy link
Contributor

I forgot that rebasing a fork has some additional steps. I'd say lets just merge this 😁
But still I'd like @markstory comments on that.

@jamisonbryant jamisonbryant force-pushed the feature/before-scope-interface-2.x branch from de525c1 to c1bf244 Compare November 10, 2023 16:04
Co-authored-by: ADmad <ADmad@users.noreply.github.com>
@markstory markstory added this to the 2.x milestone Nov 10, 2023
@markstory markstory merged commit 94fadea into cakephp:2.next Nov 10, 2023
7 of 8 checks passed
@jamisonbryant jamisonbryant deleted the feature/before-scope-interface-2.x branch January 22, 2024 19:50
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.

4 participants