🌐 AI搜索 & 代理 主页
Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Nov 6, 2019

⚠️ DO NOT MERGE (YET) ⚠️

PR for tracking changes for the 9.3.3 release. Target release date: Monday November 11.

  • Add changelog for 9.3.3 - PR Changelog for PHPCompatibility 9.3.3 #925
  • Merge this PR
  • Add release tag against master (careful, GH defaults to develop!) & copy & paste the changelog to it
  • Close the 9.3.3 milestone
  • Open a new milestone for the next release
  • If any open PRs which were milestoned for 9.3.3 do not make it into the release, update their milestone. (not applicable)
  • Tweet about the release.
  • Post about it in Slack. (WordPress #tide channel and any others people deem relevant)
  • Open PR to merge master into develop to get rid of the This branch is out-of-date with the base branch messages on each release. (only after major releases)

jrfnl and others added 13 commits October 23, 2019 01:32
Introduced in 751 / 9.1.0.

While interfaces can declare methods with the same name as the interface, as the class in which the method will be implemented will be called differently, those methods will never be seen as constructors, so shouldn't trigger this error.

Mea culpa.

Related: squizlabs/PHP_CodeSniffer 2663
Looking at the PHP source code, it appears these have been in PHP since 7.2.0, but are not documented in the PHP manual.

Ref:
* https://github.com/php/php-src/blob/master/ext/sodium/libsodium.c
ForbiddenNames: Add missing reserved keywords to ForbiddenNames sniff
…ore-sodium-constants

NewConstants: account for yet more undocumented Sodium constants
…s-should-not-apply-to-interfaces

RemovedPHP4StyleConstructors: should not apply to interfaces
Includes adding some missing "Fixes" to the 9.3.2 changelog. (already fixed up in the tag description)
@jrfnl jrfnl added Type: meta Repo strategy/structure related PR: ready for review labels Nov 6, 2019
@jrfnl jrfnl added this to the 9.3.3 milestone Nov 6, 2019
@jrfnl jrfnl requested a review from wimg November 6, 2019 20:10
@jrfnl jrfnl merged commit 1af08ca into master Nov 11, 2019
@jrfnl
Copy link
Member Author

jrfnl commented Nov 11, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Type: meta Repo strategy/structure related

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants