Skip to content

fix(config): handle shebang properly#21158

Merged
sapphi-red merged 2 commits intomainfrom
fix/handle-shebang-in-config-file
Nov 27, 2025
Merged

fix(config): handle shebang properly#21158
sapphi-red merged 2 commits intomainfrom
fix/handle-shebang-in-config-file

Conversation

@sapphi-red
Copy link
Member

@sapphi-red sapphi-red added the p2-edge-case Bug, but has workaround or limited in scope (priority) label Nov 26, 2025
@bluwy
Copy link
Member

bluwy commented Nov 26, 2025

I'm a bit confused of the original issue. Why would someone add a shebang to the config file?

@sapphi-red
Copy link
Member Author

I think it is to run vite by ./vite.config.ts. To do that, you would need to add #!/usr/bin/env -S node ./node_modules/vite/bin/vite.js -c to the config.

@bluwy
Copy link
Member

bluwy commented Nov 27, 2025

Hmm, feels a bit strange of a pattern but I guess it's not a lot of work to support it.

@mostafa-vita

This comment was marked as spam.

@sapphi-red sapphi-red merged commit df5a30d into main Nov 27, 2025
27 checks passed
@sapphi-red sapphi-red deleted the fix/handle-shebang-in-config-file branch November 27, 2025 03:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

p2-edge-case Bug, but has workaround or limited in scope (priority)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants