Hein Meling
Hein Meling
Another idea came to mind related to this. We could add to `score.proto` either - A `boolean aboveLimit` which would allow tests like this `score.getAboveLimit()`. - Or `int32 MinScore` which...
Correction: The limit / minimal score doesn't apply to each individual score object, but to the aggregate score. The aggregate score is currently in `ag.proto:Submission`. Maybe we need a `minScore`...
Here is the proto message corresponding to the json output generated by `go test -json`: ```proto message GoTestInfo { uint64 ID = 1; google.protobuf.Timestamp Time = 2; string Action =...
A recent proposal has been raised to add [parsing support for the `go test -json` output](https://github.com/golang/go/issues/53893). We should wait for this to land or help it get landed.
There is [this issue](https://github.com/golang/go/issues/43936) that would help simplify our score functions.
The teacher CLI tool should have subcommands to - [x] download (clone) a list of repositories (based on user names or group names), - [x] download (clone) all repositories and...
For testing CLI: I came across this [go-cmdtest](https://github.com/google/go-cmdtest) library from google. This appears somewhat similar to the Go command's [test script language](https://pkg.go.dev/rsc.io/script) for which there is a port here [rogpeppe/go-internal/testscript](https://github.com/rogpeppe/go-internal/)....
Much of this is already done in dat520 course repo, only need to move it to this or another tool repo.
Seems zap does not support logrotate, but there is another package that can be used. See the [FAQ](https://github.com/uber-go/zap/blob/v1.21.0/FAQ.md#does-zap-support-log-rotation).
This could be reported via some notification thing in the frontend. All teachers should be notified. The notification should be removed when the backend error disappears.