feat: pass astro version as argument to integration hook astro:config...

[feelixe]

Changes

• Pass constant ASTRO_VERSION as argument to integration hook astro:config:setup.

Testing

None added yet.

Docs

Argument needs to be added to docs page Integration API

/cc @withastro/maintainers-docs for feedback!

[changeset-bot]

⚠️ No Changeset found

Latest commit: 00c2908

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

[codspeed-hq]

CodSpeed Performance Report

Merging #13116 will not alter performance

_{Comparing feelixe:feat/pass-astro-version-to-integrations (00c2908) with main (2a85bd7)}

Summary

✅ 6 untouched benchmarks

[florian-lefebvre]

Thanks for kicking this off! A few comments:

1. I think the property should be named version
2. I'm not sure it's worth adding some tests since it will default to development
3. It needs a minor changeset
4. It needs a docs PR
5. I think to make this feature useful, we'll need a utility in https://astro-integration-kit.netlify.app/

[feelixe]

Thanks for kicking this off! A few comments:

1. I think the property should be named version
2. I'm not sure it's worth adding some tests since it will default to development
3. It needs a minor changeset
4. It needs a docs PR
5. I think to make this feature useful, we'll need a utility in https://astro-integration-kit.netlify.app/

thank you, ill get on that

[florian-lefebvre]

Do not invest too much time into it just yet, we are discussing it and think the API could take another shape (eg. exporting a constant from a new entrypoint)

[florian-lefebvre]

Thanks for the initative! Closing, more details on the roadmap discussion