gpt-engineer icon indicating copy to clipboard operation
gpt-engineer copied to clipboard

Pre commit setup

Open patillacode opened this issue 2 years ago • 1 comments

Closes #57

Please find the necessary changes to setup pre-commit in the project.

Also, I added the .github/CONTRIBUTING.md file where the installation and usage of pre-commit is explained in detail along side some other information regarding contributions.

patillacode avatar Jun 16 '23 01:06 patillacode

@AntonOsika please give this a go, I would like your approval before I merge!

Let me know if you have any feedback.

patillacode avatar Jun 16 '23 11:06 patillacode

Looks good to me!

Good job @patillacode

AntonOsika avatar Jun 17 '23 11:06 AntonOsika

Hey – how do we make these run?

Would you be able to add a .git/hooks file as well? (don't remember how to do it exactly, feel free to ask chatgpt:)

On Fri, Jun 16, 2023 at 1:42 PM Patilla Code @.***> wrote:

@AntonOsika https://github.com/AntonOsika please give this a go, I would like your approval before I merge!

Let me know if you have any feedback.

— Reply to this email directly, view it on GitHub https://github.com/AntonOsika/gpt-engineer/pull/61#issuecomment-1594544128, or unsubscribe https://github.com/notifications/unsubscribe-auth/ABCCSUL4NYKADMOEFZE4C7TXLRBB5ANCNFSM6AAAAAAZISTYGI . You are receiving this because you were mentioned.Message ID: @.***>

-- Anton Osika

AntonOsika avatar Jun 18 '23 07:06 AntonOsika

Hi @AntonOsika

I'd recommend you pull the latest main and run make dev-install This will reset your venv, reinstalling requirements and installing pre-commit

Alternatively, you can just make sure all requirements are installed and run pre-commit install

Automatically you will get the hooks when doing any commit.

Please let me know if this is working for you, also, there is documentation on how to install and a running example in CONTRIBUTING.md

patillacode avatar Jun 18 '23 11:06 patillacode