search

oss2024hnu / coursegraph-py

BSD 2-Clause "Simplified" License
5 stars 71 forks source link

이제 PR 다시 넣어주세요 #553

Open kyagrd opened 2 weeks ago

kyagrd commented 2 weeks ago

make test 에서 지금 

Error: out/ai_G.dot: syntax error in line 8
make: [Makefile:28: out/ai_G.dot] 오류 1 (무시됨)

잘못된 dot 파일을 만들어내고 있습니다.

dot를 고치려는 분들은 잘못되도 넘어간다고 넘어가면 안됩니다.

kyagrd commented 2 weeks ago

dot를 안 건드리는 분들을 위해서 저걸 그냥 넘어가게 만든거지 dot를 고치면서 dot를 엉터리로 생성하고 테스트 안하는 건 뭐하자는 건가요? 진짜 해도 해도 너무하군요

kyagrd commented 2 weeks ago

dot 관련해서 건드리는 분들은 make dottest를 꼭 실행해 보기 바랍니다.

dot 건드리려면 graphviz는 깔아놓고 개발환경은 세팅해 놓고 작업해야 되는거 아닙니까 기본적아로

어떻게 테스트도 안하고 코드를 PR 하나요 상식에 어긋나는 프로젝트 활동은 하지 밥시다.