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

The Road to 2.1 #97

Open
1 of 9 tasks
Crell opened this issue Sep 3, 2024 · 3 comments
Open
1 of 9 tasks

The Road to 2.1 #97

Crell opened this issue Sep 3, 2024 · 3 comments

Comments

@Crell
Copy link
Collaborator

Crell commented Sep 3, 2024

A new PHP release is coming soon, so it's time to get down and catch up on PER-CS! My expectation is that the next release will be 2.1, not 3.0 (though discussion may shift that).

This is a meta issue to track outstanding tasks for 2.1. We can release 2.1 when everything below has been completed, rejected, or postponed.

Please do NOT discuss details of any of these issues on this thread. Discuss them in their own threads. Only comment here with items that should be added to the consideration list.

New language functionality

Existing clarifications

Proposals

(That something is listed here doesn't mean it will be included, just that we should make a decision on it one way or another before release.)

@Crell
Copy link
Collaborator Author

Crell commented Sep 3, 2024

The full list of implemented RFCs for 8.4 is here: https://wiki.php.net/rfc#php_84

I don't think anything other than the items above warrant our attention. If you disagree, suggest something.

@TimWolla
Copy link
Contributor

TimWolla commented Sep 4, 2024

I don't think anything other than the items above warrant our attention. If you disagree, suggest something.

Looking at https://wiki.php.net/rfc/exit-as-function, it might make sense to recommend always using exit(); with parentheses.

If tools would automatically rewrite this, then code like: https://3v4l.org/KnX6u would not be confusing any more:

<?php

function test(string $param) {
    var_dump($param);
}

define('exit', 'xxx');

test(exit);

@KorvinSzanto
Copy link
Contributor

Edited to add #98

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

No branches or pull requests

3 participants