Skip to content

Conversation

@leogermani
Copy link
Contributor

Fixes the declaration of a class method to match parent class and avoid PHP Warnings and Fatals

Testing instructions:

  • Verify all tests are passing, including PHP Nightly

@leogermani leogermani added the [Status] Needs Review This PR is ready for review. label Aug 19, 2020
@leogermani leogermani added this to the 8.9 milestone Aug 19, 2020
@leogermani leogermani self-assigned this Aug 19, 2020
@github-actions github-actions bot added the [Status] Needs Package Release This PR made changes to a package. Let's update that package now. label Aug 19, 2020
@jetpackbot
Copy link
Collaborator

Warnings
⚠️ "Proposed changelog entry" is missing for this PR. Please include any meaningful changes
⚠️ The Privacy section is missing for this PR. Please specify whether this PR includes any changes to data or privacy.

This is an automated check which relies on PULL_REQUEST_TEMPLATE. We encourage you to follow that template as it helps Jetpack maintainers do their job. If you think 'Testing instructions' or 'Proposed changelog entry' are not needed for your PR - please explain why you think so. Thanks for cooperation 🤖

E2E results is available here (for debugging purposes): https://jetpack-e2e-dashboard.herokuapp.com/pr-16900

Generated by 🚫 dangerJS against aeafc79

@kraftbj kraftbj added [Type] Bug When a feature is broken and / or not performing as intended [Pri] Normal [Status] Ready to Merge Go ahead, you can push that green button! and removed [Status] Needs Review This PR is ready for review. labels Aug 19, 2020
@kraftbj kraftbj changed the title Fix method declaration compatibility Compat Package: Fix method declaration compatibility Aug 19, 2020
@kraftbj
Copy link
Contributor

kraftbj commented Aug 19, 2020

This can be considered a PHP 8.0 compat issue.

@kraftbj kraftbj merged commit 6afc747 into master Aug 19, 2020
@kraftbj kraftbj deleted the fix/method-compat branch August 19, 2020 21:54
@matticbot matticbot added [Status] Needs Changelog and removed [Status] Ready to Merge Go ahead, you can push that green button! labels Aug 19, 2020
davidlonjon added a commit that referenced this pull request Aug 20, 2020
* master: (23 commits)
  Premium Blocks: set blocks availability (#16898)
  Compat Package: Fix method declaration compatibility (#16900)
  Jetpack Dashboard: More meaningful error notices. (#16883)
  Connection REST API: Unit test for the `remote_authorize` request. (#16879)
  use blog token to request jetpack.updateBlog (#16698)
  Improve Story block media loading (#16663)
  Simplify error notices for broken connections (#16655)
  Use new heartbeat package (#16285)
  wrap-paid-block: remove component. deprecated. (#16895)
  Social Previews: improve preview description handling (#16889)
  Stats module use blog token (#16727)
  Form Block: add a new Consent Field, a new Newsletter setting, and a new newsletter variation (#16808)
  AAG: Backup card, fall back to VP content in case of /rewind API error. (#16867)
  Donations: Fix dependencies (#16892)
  Creative Mail: update option to lowercase (#16861)
  Premium Blocks: Implement the new design (#16611)
  Requests to Stats CSV use the blog token (#16716)
  Update spacing around sharing buttons to avoid no bottom margin below the customize link. (#16811)
  Jetpack SSO: Cleaning up the `requestNonce` API request. (#16830)
  Donations: Update plans when currency changes (#16844)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

[Pri] Normal [Status] Needs Package Release This PR made changes to a package. Let's update that package now. [Type] Bug When a feature is broken and / or not performing as intended

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants