Skip to content

Conversation

@florian-lefebvre
Copy link
Member

Changes

  • It has been reported several times that people with complex redirects couldn't find the failing ones
  • This PR makes it so that the corresponding error includes the origin and the destination

Testing

Should pass

Docs

Changeset

@florian-lefebvre florian-lefebvre self-assigned this Apr 24, 2025
@github-actions github-actions bot added pkg: astro Related to the core `astro` package (scope) docs pr labels Apr 24, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 24, 2025

CodSpeed Performance Report

Merging #13680 will not alter performance

Comparing feat/external-redirects-better-error (ef1f2cd) with main (6ed8360)

Summary

✅ 6 untouched benchmarks

@ematipico ematipico changed the title feat: improve external redirects message fix: improve external redirects message Apr 24, 2025
Copy link
Member

@sarah11918 sarah11918 left a comment

Choose a reason for hiding this comment

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

LGTM!

@florian-lefebvre florian-lefebvre merged commit 18e1b97 into main Apr 27, 2025
16 checks passed
@florian-lefebvre florian-lefebvre deleted the feat/external-redirects-better-error branch April 27, 2025 15:18
@astrobot-houston astrobot-houston mentioned this pull request Apr 28, 2025
openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants