Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: handle bun projects in apheleia-npx #292

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ink-pot-monkey
Copy link

The apheleia-npx script now checks for bun projects and handles them correctly by using the installed bun bin exec the command.

@ink-pot-monkey
Copy link
Author

Not sure why the formatter test is failing, as this change shouldn't have any impact on previous formatters.

Would it be useful for me to add in a test case here for bun?

@raxod502
Copy link
Member

image

Could be that a newer version of Prettier formats differently by default, breaking the test, or a bug in Prettier being able to find configuration files correctly. Probably the former. I can look into it in a bit.

A new test would be great although the test framework might need to be tweaked a bit to write a good test for this. That is also something I can look at in a bit.

Or you can have a go if you feel up to it earlier.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants