Skip to content

testing Declare strict types #201

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.

Sign up for GitHub

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

Jump to bottom
Closed
szepeviktor wants to merge 11 commits into from
Closed

testing Declare strict types #201

szepeviktor wants to merge 11 commits into from

Conversation

@szepeviktor
Copy link
Collaborator

@szepeviktor szepeviktor commented Mar 14, 2024

🥼

@szepeviktor szepeviktor self-assigned this Mar 14, 2024
@szepeviktor
Copy link
Collaborator Author

szepeviktor commented Mar 14, 2024 •
edited
Loading

@iamgergo The result is:
$model->getKey() (sometimes) returns an integer, not a string.

It is 💯% up to you.

@szepeviktor szepeviktor removed their assignment Mar 14, 2024
@szepeviktor
Merge branch 'master' into szepeviktor-patch-1
2022348
@szepeviktor
parse_url may return null
d65385e 
> If the requested component doesn't exist within the given URL, null will be returned.
@szepeviktor
Update Repeater.php
fecfeae
@iamgergo
Copy link
Member

iamgergo commented Aug 11, 2025

@szepeviktor I'm open to add declare strct types to every file. I can configure pint to add it right?

@szepeviktor szepeviktor deleted the szepeviktor-patch-1 branch August 11, 2025 07:49
@szepeviktor szepeviktor mentioned this pull request Aug 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@szepeviktor @iamgergo