memfreeme / memfree

MemFree - Hybrid AI Search Engine & AI Generate UI
https://www.memfree.me
MIT License
785 stars 101 forks source link

[Feature Request] Adding pre-commit hooks #48

Closed sank8-2 closed 50 minutes ago

sank8-2 commented 2 hours ago

Is there an existing issue for this?

Feature Description

Addition of pre-commit hooks for linters and code formatting.

Use Case

Code optimisation

Record

github-actions[bot] commented 2 hours ago

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊

You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

ahaapple commented 2 hours ago

@sank8-2 Great idea. Are you willing to do this job?

sank8-2 commented 2 hours ago

Yes, please assign it

sank8-2 commented 1 hour ago

@ahaapple Please check it

github-actions[bot] commented 49 minutes ago

Hello @sank8-2! Your issue #48 has been closed. Thank you for your contribution!

ahaapple commented 37 minutes ago

@sank8-2 Hi, After I pulled the latest code, I found that the pre-commit check was not automatically executed.

  1. Can you update the pre-commit related operations in https://github.com/memfreeme/memfree/blob/main/Contribution.md?

  2. Can you better configure the pre-commit config for the nextjs project?

Thanks

ahaapple commented 32 minutes ago

These are two related issues:

https://github.com/memfreeme/memfree/issues/51

https://github.com/memfreeme/memfree/issues/50

sank8-2 commented 28 minutes ago

Ok