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

Run pre-commit checks in GitHub Actions #20

Merged
merged 2 commits into from
Jun 13, 2024
Merged

Run pre-commit checks in GitHub Actions #20

merged 2 commits into from
Jun 13, 2024

Conversation

wjt
Copy link
Member

@wjt wjt commented Jun 13, 2024

No description provided.

@wjt wjt requested a review from wnbaum June 13, 2024 13:59
Copy link
Contributor

@wnbaum wnbaum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cool! Looks good to me. At some point we should figure out how to change the max line length on godot-gdscript-toolkit's gdlint. There was an option in gdformat, but not gdlint... After not being able to commit for an hour because my paths were too long I ended up just removing gdlint lol. Maybe I am just silly for wanting longer line lengths though?

@wnbaum wnbaum merged commit 917f69f into main Jun 13, 2024
1 check passed
@wnbaum wnbaum deleted the pre-commit-action branch June 13, 2024 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants