commit | 5b046ec0c510d59c8b72bd99e236313a4d7e5fa2 | [log] [tgz] |
---|---|---|
author | Han-Wen Nienhuys <hanwen@google.com> | Fri Nov 15 20:26:32 2019 -0800 |
committer | Han-Wen Nienhuys <hanwen@google.com> | Fri Nov 15 20:26:32 2019 -0800 |
tree | c919f6933082eaaa28e4aadc8731add8749f539a | |
parent | b00f876f20619fae72035797b63b5b957bccc6f7 [diff] |
cmd/checker: provide more diagnostics Change-Id: I0cacef6b68bb529869d5cd2f6882bc3c93fc17f0
This is a style verifier intended to be used with the Gerrit checks plugin.
go install github.com/bazelbuild/buildtools/buildifier curl -o google-java-format.jar https://github.com/google/google-java-format/releases/download/google-java-format-1.7/google-java-format-1.7-all-deps.jar
Obtain an HTTP password, and put it in testsite-auth
. The format is username:secret
.
Register a checker
go run ./cmd/checker -auth_file=testsite-auth --gerrit http://localhost:8080 \ --language go --repo gerrit --register
go run ./cmd/checker -auth_file=testsite-auth --gerrit http://localhost:8080 \ --list
go run ./cmd/checker -auth_file=testsite-auth --gerrit http://localhost:8080
For simplicity of deployment, gerritfmt checker is stateless. All the necessary data is encoded in the checker UUID
handle file types (symlink) and deletions
more formatters: clang-format, typescript, jsformat, ... ?
isolate each formatter to run with a separate gvisor/docker container.
tests: the only way to test this reliably is to spin up a gerrit server, and create changes against the server.
Update the list of checkers periodically.
This currently runs the formatters without sandboxing. Critical bugs (heap overflow, buffer overflow) in formatters can be escalated to obtain the OAuth2 token used for authentication.
The currently supported formatters are written in Java and Go, so this should not be an issue.
This is not an official Google product