Baseflow / Baseflow/flutter_cache_manager
[2.0.0] Rename Config to CacheManagerConfig
- 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
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