CLI 로깅 추가 및 Datahub GMS 연결체크 기능 분리 #95

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.

Sign up for GitHub

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

Jump to bottom

Open

ParkGyeongTae wants to merge 1 commit into master from feature/94-add-logging-divide-gms-server-check

Contributor

ParkGyeongTae commented May 3, 2025

#️⃣ Issue Number

CLI 로깅 추가 및 Datahub GMS 연결체크 기능 분리 #94

📝 요약(Summary)

CLI 로깅 추가 및 Datahub GMS 연결체크 기능 분리
lang2sql 실행시 실행과정을 로깅으로 나타내 동작을 확인할 수 있도록 수정
DataHub GMS 서버 연결부분을 메인 로직에서 분리하여, 단일 기능만 수행하도록 수정

💬 To Reviewers (선택)

없음

PR Checklist

lang2sql --datahub_server http://34.125.222.80:8080 run-streamlit

reference) How to Code Review

따봉(👍): 리뷰어가 리뷰이의 코드에서 칭찬의 의견을 남기고 싶을 때 사용합니다.
느낌표(❗): 리뷰어가 리뷰이에게 필수적으로 코드 수정을 요청할 때 사용합니다.
물음표 (❓): 리뷰어가 리뷰이에게 의견을 물어보고 싶을 때 사용합니다.
알약 (💊): 리뷰어가 리뷰이의 코드에서 개선된 방법을 제안하지만 그것의 반영이 필수까지는 아닐 때 사용합니다.


          fix: add logging and server checker

cc60e43

ParkGyeongTae self-assigned this

ParkGyeongTae added this to the Mission: Version 1.0.0 Complete ✅ milestone

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet