fix(astro): add define:vars to HTMLAttributes

[ZerdoX-x]

Changes

• This adds the missing define:vars to the HTMLAttributes type as explained the same in fix(astro): add class:list to HTMLAttributes #5284

Testing

Was not tested.

Docs

The docs don't explicitly mention any props of HTMLAttributes.

[changeset-bot]

🦋 Changeset detected

Latest commit: 38d28b8

The changes in this PR will be included in the next version bump.

Not sure what this means? Click here to learn what changesets are.

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

[ZerdoX-x]

Not sure why this is not fixed yet or even implemented initially.

[Princesseuh]

Hello, thank you for this contribution! However, it seems like there might have been a misunderstanding on your end on how define:vars work 😅

The define:vars attribute is only available on style and script, hence why you get an error when using it somewhere else. You can see more information about the directive on our docs

Your PR also wouldn't fix this, as it updates the HTMLAttributes type that people can use manually, not the one that is used for checking the template. define:vars is also not on the interface you're omitting from, so it would just do nothing. When used manually HTMLAttributes<'script'> properly has the define:vars attribute, as it should

[ZerdoX-x]

Oh.
That's cool. That is actually more convenient. Sorry for my misunderstanding and thank you for your response <3