Skip to content

[13.x] Fix ValidationImageFileRuleTest#60348

Merged
taylorotwell merged 2 commits into
laravel:13.xfrom
jackbayliss:testDimensionsWithCustomImageSizeMethod-fun
Jun 2, 2026
Merged

[13.x] Fix ValidationImageFileRuleTest#60348
taylorotwell merged 2 commits into
laravel:13.xfrom
jackbayliss:testDimensionsWithCustomImageSizeMethod-fun

Conversation

@jackbayliss
Copy link
Copy Markdown
Contributor

@jackbayliss jackbayliss commented Jun 1, 2026

Was surprised to see this failing via https://github.com/laravel/framework/actions/runs/26788085152/job/78968350449

It's because the vars it had were removed in #60338 so now it's doomed to fail.

This PR basically makes it simpler... (I reckon) and adds a comment to ensure it's not changed again.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Jun 1, 2026

Thanks for submitting a PR!

Note that draft PRs are not reviewed. If you would like a review, please mark your pull request as ready for review in the GitHub user interface.

Pull requests that are abandoned in draft may be closed due to inactivity.

@jackbayliss jackbayliss marked this pull request as ready for review June 1, 2026 23:47
@taylorotwell taylorotwell merged commit 4ada21a into laravel:13.x Jun 2, 2026
54 checks passed
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

Successfully merging this pull request may close these issues.

2 participants