Baseflow / Baseflow/flutter_cache_manager

[2.0.0] Rename Config to CacheManagerConfig

Đang mở
#237 1 bình luận 1 reaction 0 người được giao Xem trên GitHub
Ngôn ngữ chính
Dart
Star
809
Fork
510
Merge trung bình
6 phút
Pull request đã merge (30 ngày)
1

Mô tả

## 🏗 Enhancement Proposal

Since version 2.0.0 the flutter_cache_manager exports a class called `Config`. This name is very general and causes problems in all my projects because I have a general Config class.

I think the name `Config` is a very bad choice, as it is way to general. So I propose renaming it to `CacheManagerConfig` to prevent conflicts like these to occur.

I know that it is possible to work around this issue by using `import a as b`, but I think this should be avoided to make the development process more easy.

Also the name `CacheManagerConfig` would be much more intuitive to use, as you usually wouldn't expect to have a class called `Config`, but rather somthing like `CacheManager***` while writing your code with intellisense.

Hướng dẫn đóng góp

Mở hướng dẫn đóng góp

Hướng nghiên cứu

Bắt đầu bằng cách xác định khai báo Config được export và mọi tham chiếu đến nó trong flutter_cache_manager. Xem xét các bài kiểm thử hiện có của package và các điểm vào API công khai trước khi quyết định phạm vi đổi tên; công việc được xem là hoàn tất khi tên CacheManagerConfig được yêu cầu được sử dụng nhất quán và các bài kiểm thử bị ảnh hưởng đều chạy đạt.

Do mô hình lập chỉ mục viết ra từ nội dung của issue.

Đánh giá

Công nghệ
dart, flutter
Lĩnh vực
mobile
Loại issue
Tái cấu trúc
Độ khó
4/5
Thời gian dự kiến
3-5 ngày
Mức độ hoạt động
Đình trệ
Độ rõ ràng
Khá rõ ràng
Mức phù hợp với người mới
35/100

Nhận issue mới trong hộp thư của bạn

Bản tóm tắt ngắn những issue GitHub phù hợp với người mới.