JOJ/JOJ3
24
5
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

clang-tidy parser and executor #26

Merged
张泊明518370910136 merged 26 commits from clang-tidy into master 2024-05-18 02:50:13 +08:00
Conversation 25 Commits 26 Files Changed 9 -11
1 changed files with 0 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 87c2bc38d0 - Show all commits

11
internal/parsers/clang_tidy/parser.go
View File

@ -1,8 +1,6 @@
package clang_tidy package clang_tidy
import ( import (
"encoding/json"
"os"
"strings" "strings"
"focs.ji.sjtu.edu.cn/git/FOCS-dev/JOJ3/internal/stage" "focs.ji.sjtu.edu.cn/git/FOCS-dev/JOJ3/internal/stage"
@ -28,15 +26,6 @@ func Parse(executorResult stage.ExecutorResult, conf Conf) stage.ParserResult {
messages := parse_lines(lines, conf) messages := parse_lines(lines, conf)
formatted_messages := format(messages) formatted_messages := format(messages)
// TODO: Handle the json file (parse into markdown and delete it?)
json_file, _ := os.Create("./clangtidy_result.json")
defer json_file.Close()
encoder := json.NewEncoder(json_file)
encoder.SetEscapeHTML(false)
encoder.SetIndent("", " ")
_ = encoder.Encode(formatted_messages)
// TODO: Handle unexpected errors from executor // TODO: Handle unexpected errors from executor
// if executorResult.Status != stage.Status(envexec.StatusAccepted) { // if executorResult.Status != stage.Status(envexec.StatusAccepted) {
zjc_he marked this conversation as resolved Outdated
张泊明518370910136 commented 2024-05-07 15:02:07 +08:00
Outdated
Review
Copy Link

I think we can uncomment it.

I think we can uncomment it.
张佳澈520370910044 commented 2024-05-07 15:44:01 +08:00
Outdated
Review
Copy Link

No, as long as the clang-tidy report is not empty, its return value is 1 and the executor result status is Nonzero Exit Status:

main_test.go:117: actual[1].Results[0].Comment = Unexpected executor status: Nonzero Exit Status.
        Stderr: 17855 warnings and 1 error generated.
        Error while processing /w/src/sillycode.cpp.
No, as long as the clang-tidy report is not empty, its return value is 1 and the executor result status is Nonzero Exit Status: ``` main_test.go:117: actual[1].Results[0].Comment = Unexpected executor status: Nonzero Exit Status. Stderr: 17855 warnings and 1 error generated. Error while processing /w/src/sillycode.cpp. ```
张泊明518370910136 commented 2024-05-07 15:53:48 +08:00
Outdated
Review
Copy Link

Ok, then we can assume clang-tidy do not have unexpected errors for now. clang-tidy built in rules based on Libtooling are usually very strong. An alternative approach can be checking executorResult.exitStatus if we know it will only return 0 and 1.

Ok, then we can assume clang-tidy do not have unexpected errors for now. `clang-tidy` built in rules based on Libtooling are usually very strong. An alternative approach can be checking `executorResult.exitStatus` if we know it will only return 0 and 1.
// return stage.ParserResult{ // return stage.ParserResult{
zjc_he marked this conversation as resolved Outdated
张泊明518370910136 commented 2024-05-07 02:48:08 +08:00
Outdated
Review
Copy Link

Do we still need this file?

Do we still need this file?
张佳澈520370910044 commented 2024-05-07 09:57:00 +08:00
Outdated
Review
Copy Link

I don't know. We will need this data for drone, but I don't know where to put it for now.

I don't know. We will need this data for drone, but I don't know where to put it for now.
张泊明518370910136 commented 2024-05-07 09:58:37 +08:00
Outdated
Review
Copy Link

I think running JOJ3 will be the only step in the future. It just provides coast-to-coast experience.

I think running JOJ3 will be the only step in the future. It just provides coast-to-coast experience.
张佳澈520370910044 commented 2024-05-07 10:02:19 +08:00
Outdated
Review
Copy Link

What do you mean? JOJ3 will cover everything and we don't need drone anymore?

This data shows the details for all cases, we may need to parse them into markdown in the future, so if we don't need to save it into a file I'll just put it there and wait for future needs

What do you mean? JOJ3 will cover everything and we don't need drone anymore? This data shows the details for all cases, we may need to parse them into markdown in the future, so if we don't need to save it into a file I'll just put it there and wait for future needs
张泊明518370910136 commented 2024-05-07 10:04:43 +08:00
Outdated
Review
Copy Link

Yes. Drone/Gitea actions will only be used to trigger the running of JOJ3, and JOJ3 should be the only step. We can just leave the details in logs, so if any debug purpose, student can check the output in drone/gitea actions to see. So that we can only give simple and brief feedback and gitea issue comment, which should be enough for most cases.

Yes. Drone/Gitea actions will only be used to trigger the running of JOJ3, and JOJ3 should be the only step. We can just leave the details in logs, so if any debug purpose, student can check the output in drone/gitea actions to see. So that we can only give simple and brief feedback and gitea issue comment, which should be enough for most cases.