Baseflow / Baseflow/flutter_cache_manager
[2.0.0] Rename Config to CacheManagerConfig
- Langage dominant
- Dart
- Étoiles
- 809
- Forks
- 510
- Merge moyen
- 6 min
- PR mergées (30 j)
- 1
Description
## 🏗 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.
Guide de contribution
Ouvrir le guide de contribution
Piste de recherche
Start by locating the exported Config declaration and every reference to it in flutter_cache_manager. Review the package's existing tests and public API entry points before deciding the rename scope; done means the requested CacheManagerConfig name is used consistently and the affected tests pass.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Évaluation
- Stack technique
- dart, flutter
- Domaine
- mobile
- Type d'issue
- Refactorisation
- Difficulté
- 4/5
- Temps estimé
- 3-5 jours
- Activité
- À l'abandon
- Clarté
- Plutôt claire
- Accessibilité débutants
- 35/100