The `is_login` status check seems unnecessary for the `search_train` function.
Nitro1231 opened this issue · 2 comments
Nitro1231 commented
우선 라이브러리 개발 노고에 감사드립니다. 덕분에 잘 사용하고 있습니다! (_ _)
Lines 181 to 182 in 758ba02
srt.py
스크립트의 search_train
함수의 기능을 고려할때, 181~182번째 줄에 있는 is_login
확인문이 꼭 필요해 보이지 않습니다. 더 나아가 유저의 로그인 정보 없이 열차 정보를 가져오는 기능이 있다면 그냥 열차 정보만 필요할때 유용할듯 합니다. 혹시 로그인 확인문을 넣으신 이유가 있으실까요?
ryanking13 commented
안녕하세요. 그렇네요, 검색은 굳이 로그인 체크가 필요하지 않을 것 같습니다. 관련하여 수정하는 PR을 올리는 데에 관심이 있으실까요?
ryanking13 commented
Closing as resolved by #238