Automate pre-commit testing with hook

[godot-rust]

Automate pre-commit testing with hook

[glfmn]

Installation

preferred with ln

$ ln -s ../../hooks/pre-commit.sh .git/hooks/pre-commit
$ chmod +x .git/hooks/pre-commit

or with cp

$ cp hooks/pre-commit.sh .git/hooks/pre-commit
$ chmod +x .git/hooks/pre-commit

Using ln vs cp

• ln: if you happen to checkout a part of the history which does not include the pre-commit hook in the root, it breaks because the file it links to is no longer present, requiring manual testing and --no-verify
• cp: you will need to manually apply updates by copying updated hooks

[nical]

Nice!

Godot can't find libgdnative_test.so

libgdnative_test.so is in target/debug/ after building the test/ directory. Right now it has to be copied into the godot project of the test directory which is begging to be automated.

Godot exits status 0 on failure to find libgdnative_test.so

Oops! that's a mistake and thankfully an easy one to fix, thanks for spotting this. See https://github.com/GodotNativeTools/godot-rust/blob/master/test/project/main.gd#L16 this branch should set the exit code but currently doesn't.

[glfmn]

Thanks for getting back to me, I have no idea how I didn't notice that it was the file that's mentioned in this line on your CONTRIBUTING.md file!

cp ./target/debug/libgdnative_test.so ./test/project/lib

It's in my script, copied from your CONTRIBUTING.md file, it seems to have been renaming the file lib instead of putting it in the folder test/project/lib/. I updated the line accordingly and it works now.

I've opened a PR: #65.