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

API Rescue Master Branch PR: Support new extend*() format for extension methods #371

Conversation

GuySartorelli
Copy link
Member

@GuySartorelli GuySartorelli force-pushed the pulls/2/rescue-master-new-extension-functionality branch from e106638 to f7bcf58 Compare August 23, 2022 01:36
@GuySartorelli GuySartorelli force-pushed the pulls/2/rescue-master-new-extension-functionality branch from efefea9 to 9eb2aee Compare August 26, 2022 06:09
@emteknetnz
Copy link
Member

Re-running tests now that dependent PR has been merged

@GuySartorelli
Copy link
Member Author

Needs silverstripe/silverstripe-asset-admin#1290 and silverstripe/silverstripe-postgresql#135 to be merged for CI to go green.

@emteknetnz
Copy link
Member

Merged both, rerunning tests

@emteknetnz
Copy link
Member

Still failing

@GuySartorelli
Copy link
Member Author

The current CI failure will be resolved when silverstripe/silverstripe-framework#10479 is merged (though there will likely be others after that)

@emteknetnz emteknetnz merged commit 71d22bb into silverstripe:2 Sep 1, 2022
@emteknetnz emteknetnz deleted the pulls/2/rescue-master-new-extension-functionality branch September 1, 2022 02: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.

2 participants