-
Notifications
You must be signed in to change notification settings - Fork 20
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
컴파일 기능 #144
컴파일 기능 #144
Conversation
62d30d8
to
0fa2656
Compare
https://github.com/AviFS/tio-api async TIO.run(code, input, lang) -> Promise([stdout, stderr+timing, ?]) |
fba8f90
to
240bf19
Compare
아래 GIF처럼 기능은 어느정도 동작하도록 구현 완료하였는데, 몇 가지 불안정한 문제가 있습니다.
@smartwe 가능하시면 위 내용 중에서 TIO 전송 시 깨지는 문제와 나머지 항목들에 대해서 의견 부탁드립니다. |
아 그리고 입력을 수동으로 입력받게하는건 어떤가요? |
BOJ 에서 빌드하는 환경과 완전히 동일하게 세팅할수가 없어서 이슈가 종종 있을 것 같네요. 아쉽지만 closed 하겠습니다 😥 |
|
5e65bdf
to
356714c
Compare
* feat: add compile feature (except dropdown) * chore: lint and pretty correct * feat: add exception of unsupported language * Update src/js/features/compile.js --------- Co-authored-by: Joona Yoon <joonas-yoon@users.noreply.github.com>
(cherry picked from commit e603758)
(cherry picked from commit 65d66b2)
(cherry picked from commit f9e7137)
356714c
to
41ce960
Compare
@ruk13xa 덕분에 인코딩쪽 문제까지 해결하여 위처럼 통합했습니다. 감사합니다 :) 몇 가지 언어들을 테스트해봤는데, Ada 의 경우에는 BOJ 채점 환경과 버전이 달라서 결과가 다른 것 확인했습니다. 그리고 이름은 컴파일보다는 예제 테스트가 목적이므로 '테스트'로 변경했습니다. 감사합니다. |
추가로 틀렸을시 어떻게 출력되었는지도 알려주면 좋을 것 같네요 |
@ruk13xa 우선 지금 main 브랜치에 작업하신 커밋들은 체리픽으로 여기 레포지토리의 feature/compile 브랜치로 합쳤습니다. |
넵 :D |
(메모) 프로그램이 -1 을 리턴하면서 종료하는 경우, 비정상 시그널이라 런타임 오류가 맞지만 단순 출력 비교로는 정답으로 표시됨
|
연결된 이슈
아이디어 제안
내용
Base feature branch 입니다.
feature/compile
브랜치를 base로 해서 fork 한 후에 작업해주세요.Pull Request 시에는 다시 "
feature/compile
<-forked repo branch
" 형태로 요청해주시면 이 PR로 정리하겠습니다.스크린샷
아래와 같이 버튼을 만들었고, 클릭 시 작성한 코드가 넘어갑니다.
다음 코드를 채워주시면 좋겠습니다.
boj-extended/src/js/features/compile.js
Lines 34 to 38 in 3a63b38