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

Add tests/python_build to clang-format pre-commit hook ignore list #87000

Merged

Conversation

akx
Copy link
Contributor

@akx akx commented Jan 9, 2024

It had been ignored by misc/scripts/clang_format.sh since 388d35b but not the hook; running the hook on all files yields changes in GLSL files in that directory.

@akx akx requested a review from a team as a code owner January 9, 2024 12:41
@akx
Copy link
Contributor Author

akx commented Jan 9, 2024

cc @akien-mga, you seem to have been maintaining these :)

@akx
Copy link
Contributor Author

akx commented Jan 9, 2024

CI fails because of:

could not find pull request diff: HTTP 500 (https://api.github.com/repos/godotengine/godot/pulls/87000)

That's a new one...

@AThousandShips
Copy link
Member

Looks like GitHub is having server problems again

@akien-mga akien-mga changed the title Add tests/python_build to clang-format pre-commit hook ignore list Add tests/python_build to clang-format pre-commit hook ignore list Jan 9, 2024
It had been ignored by misc/scripts/clang_format.sh since 388d35b but not the hook.
@akx akx force-pushed the pre-commit-clang-format-ignore-fix branch from 4b9abe5 to de7133e Compare January 9, 2024 14:07
@akx akx requested a review from akien-mga January 9, 2024 14:07
@akx
Copy link
Contributor Author

akx commented Feb 9, 2024

Gentle ping, @akien-mga?

@akien-mga akien-mga modified the milestones: 4.x, 4.3 Feb 9, 2024
@akien-mga akien-mga merged commit d87dd6e into godotengine:master Feb 9, 2024
15 checks passed
@akien-mga
Copy link
Member

akien-mga commented Feb 9, 2024

Thanks! And congrats for your first merged Godot contribution 🎉

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

Successfully merging this pull request may close these issues.

3 participants