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

EXPERIMENTAL - add syntax check to reviewer #637

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

senko
Copy link
Collaborator

@senko senko commented Feb 7, 2024

This is experimental, do not merge.

Adds a syntax check to reviewer.

Uses prettier for JS, ruff for Python and gofmt for Go. You'll need to have the tool(s) installed in the proper environment (doesn't do any checks).

Also doesn't send the files back to the codemonkey for fixup if the syntax is incorrect. As I said, it's experimental.

@thewshii
Copy link

thewshii commented Feb 7, 2024

Never heard of go, this'll be fun. I should be set up bout to run it.

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.

None yet

2 participants