Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Docs: Use require without brackets #2337

Merged
merged 1 commit into from
Dec 20, 2024

Conversation

piotr-cz
Copy link
Contributor

@piotr-cz piotr-cz commented Dec 17, 2024

Because brackets with require is not necessary and implies it's a function, while it's a statement.

In phinx code, require is used without brackets:

$autoloader = require __DIR__ . '/../src/composer_autoloader.php';

$app = require __DIR__ . '/phinx.php';

@piotr-cz piotr-cz changed the title Docs: Use require without brackets Docs: Use require without brackets Dec 17, 2024
@MasterOdin MasterOdin merged commit 5bad109 into cakephp:0.x Dec 20, 2024
12 checks passed
@piotr-cz piotr-cz deleted the docs/require-statement branch December 22, 2024 07:47
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants