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

Allow using bun instead of node #3089

Open
gaborbernat opened this issue Dec 8, 2023 · 5 comments
Open

Allow using bun instead of node #3089

gaborbernat opened this issue Dec 8, 2023 · 5 comments

Comments

@gaborbernat
Copy link

search you tried in the issue tracker

bun

describe your actual problem

Documentation about bun: https://bun.sh.

The primary benefit would be much faster startup/run on larger projects.

pre-commit --version

latest

@asottile
Copy link
Member

asottile commented Dec 8, 2023

@gaborbernat
Copy link
Author

I'll try if I get to it, no promises 😆 would it be possible to convince https://github.com/pre-commit/mirrors-prettier#prettier-mirror to use bun without needing changes to that repo, though?

@asottile
Copy link
Member

asottile commented Dec 8, 2023

yeah in theory you'd set language: bun and it should just work assuming we make language: bun as compatible with language: node as possible

@gaborbernat
Copy link
Author

So it would be:

-   repo: https://github.com/pre-commit/mirrors-prettier
    language: bun
    rev: ''  # Use the sha / tag you want to point at
    hooks:
    -   id: prettier

?

@asottile
Copy link
Member

asottile commented Dec 8, 2023

close -- language is defined at the hook level not a the repo level

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

No branches or pull requests

2 participants