Skip to content

Conversation

@Princesseuh
Copy link
Member

@Princesseuh Princesseuh commented Oct 28, 2025

Changes

Testing

Updated tests that tested old behaviour to test new behavior, ha

Docs

withastro/docs#12639

@changeset-bot
Copy link

changeset-bot bot commented Oct 28, 2025

⚠️ No Changeset found

Latest commit: 27479ab

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added the pkg: astro Related to the core `astro` package (scope) label Oct 28, 2025
@florian-lefebvre
Copy link
Member

Does this address #13995?

@Princesseuh
Copy link
Member Author

Does this address #13995?

Yea

Copy link
Member

@florian-lefebvre florian-lefebvre left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code LGTM, this is missing a changeset which can only be done after the docs PR

@Princesseuh
Copy link
Member Author

There's a docs PR now, think I did it correctly

@Princesseuh Princesseuh merged commit dfb51d4 into next Oct 31, 2025
31 of 32 checks passed
@Princesseuh Princesseuh deleted the feat/cropping branch October 31, 2025 01:18
@florian-lefebvre
Copy link
Member

florian-lefebvre commented Oct 31, 2025

@Princesseuh FYI we don't merge code PRs until the docs PR and the changeset are done now, you'll know next time :)

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

Labels

pkg: astro Related to the core `astro` package (scope)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Do not upscale images in the default image service getImage() silently fails to resize/crop images

4 participants