Skip to content

chore(astro): todos cleanup#14405

Merged
florian-lefebvre merged 2 commits intomainfrom
chore/todos-cleanup
Sep 19, 2025
Merged

chore(astro): todos cleanup#14405
florian-lefebvre merged 2 commits intomainfrom
chore/todos-cleanup

Conversation

@florian-lefebvre
Copy link
Member

Changes

  • Looking at todos for v6, I saw some that are not relevant anymore or can be done against main directly

Testing

Should pass

Docs

N/A, refactor

@florian-lefebvre florian-lefebvre self-assigned this Sep 19, 2025
@changeset-bot
Copy link

changeset-bot bot commented Sep 19, 2025

⚠️ No Changeset found

Latest commit: 318a460

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 Sep 19, 2025
@codspeed-hq
Copy link

codspeed-hq bot commented Sep 19, 2025

CodSpeed Performance Report

Merging #14405 will not alter performance

Comparing chore/todos-cleanup (318a460) with main (f59581f)1

Summary

✅ 6 untouched

Footnotes

  1. No successful run was found on main (b249684) during the generation of this report, so f59581f was used instead as the comparison base. There might be some changes unrelated to this pull request in this report.

internalURL: 'astro/compiler-runtime',
// TODO: this is no longer necessary for `Astro.site`
// but it somehow allows working around caching issues in content collections for some tests
astroGlobalArgs: JSON.stringify(astroConfig.site),
Copy link
Member Author

Choose a reason for hiding this comment

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

Looks like it's not relevant anymore with the content layer

@florian-lefebvre florian-lefebvre marked this pull request as ready for review September 19, 2025 09:06
@florian-lefebvre florian-lefebvre merged commit 6b92b3d into main Sep 19, 2025
21 checks passed
@florian-lefebvre florian-lefebvre deleted the chore/todos-cleanup branch September 19, 2025 09:48
aprici7y added a commit to aprici7y/astro that referenced this pull request Oct 19, 2025
Restores the astroGlobalArgs compiler option that was removed in withastro#14405. This parameter is needed to pass the site config to the compiler so Astro.site is available in getStaticPaths().
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.

2 participants