chore: drop unused dependencies and unreferenced video assets - #20
Merged
Conversation
lib/ 전체를 훑어 import 가 하나도 없는 패키지를 걷어냈다. - google_ml_kit: 서브패키지 16개를 끌고 오면서 Android 빌드 시간의 상당 부분을 차지했다. 코드에서 쓰인 적이 없다 - tflite_flutter, camera, firebase_ml_model_downloader: README 의 "지원 예정" OCR 기능용으로 미리 넣어둔 것들. 실제로 필요해지는 시점에 다시 넣는다 (#9) - http: dio 로 대체됐다 (#16) - video_player, salomon_bottom_bar, fl_chart: 처음부터 쓰인 적 없음 - flutter_launcher_icons: 빌드 도구라 dev_dependencies 로 옮긴다 assets/Video 는 참조하는 코드가 없는데 17MB 가 그대로 APK 에 실리고 있었다. pubspec 의 애셋 목록에서만 빼고 파일은 저장소에 남긴다. syncfusion_flutter_charts 는 남긴다. Phone.dart 가 실제로 쓰고 있어서, 지금 빼면 화면이 깨진다. 이슈 #13 에는 fl_chart 로 단일화한다고 적었는데 확인해보니 정작 fl_chart 쪽이 미사용이었다. 차트 라이브러리 정리는 Phone.dart 가 새로 쓰이는 Phase 3 에서 함께 결정한다 (#7). 테스트 25건 통과, analyze 에러 0건.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
#21 체크리스트 중 "미사용 의존성 정리".
lib/ 전체에서 import 0인 패키지 제거: google_ml_kit, tflite_flutter, camera, firebase_ml_model_downloader, http, video_player, salomon_bottom_bar, fl_chart
flutter_launcher_icons는 빌드 도구라 dev_dependencies로 이동.
assets/Video도 쓰는 데가 없는데 17MB가 APK에 들어가고 있었음. pubspec 애셋 목록에서만 뺐고 파일은 그대로 둠.
디버그 APK 295MB → 101MB.
syncfusion_flutter_charts는 남김. Phone.dart가 실제로 쓰고 있어서 빼면 화면 깨짐. 차트 정리는 화면 다시 쓸 때 같이.
테스트 25건 통과, analyze 에러 0건.