Project lint checked and step added to config.yml

.circleci/config.yml

@@ -8,4 +8,5 @@ jobs:
       - run: flutter doctor
       - run: flutter pub get
       - run: flutter pub run build_runner build
+      - run: flutter analyze lib
       - run: flutter test

.gitignore

@@ -33,7 +33,7 @@ migrate_working_dir/
 .pub/
 /build/
 *.g.dart
-*.mock.dart
+*.mocks.dart
 
 # Symbolication related
 app.*.symbols

lib/base_viewmodel.dart

@@ -1,6 +1,6 @@
 import 'package:stacked/stacked.dart';
 
-import 'Utils/ViewState.dart';
+import 'Utils/view_state.dart';
 
 abstract class BaseViewmodel extends BaseViewModel{
 

lib/base_widget.dart

@@ -2,10 +2,10 @@ import 'package:flutter/material.dart';
 import 'package:stacked/stacked.dart';
 import 'package:toast/toast.dart';
 
-import 'BaseViewModel.dart';
+import 'base_viewmodel.dart';
-import 'Utils/Constants.dart';
+import 'Utils/constants.dart';
-import 'Utils/ViewState.dart';
+import 'Utils/view_state.dart';
-import 'Utils/ViewUtils.dart';
+import 'Utils/view_utils.dart';
 
 abstract class BaseStatelessWidget<T extends BaseViewmodel>
     extends StatelessWidget {
@@ -16,6 +16,7 @@ abstract class BaseStatelessWidget<T extends BaseViewmodel>
   Widget displayWidget(BuildContext context, T model, Widget? child);
 
   @override
+  // ignore: avoid_renaming_method_parameters
   Widget build(BuildContext parent) {
     ToastContext().init(parent);
     return Scaffold(

lib/data/network/app_dio.dart

@@ -1,3 +1,4 @@
+// ignore: depend_on_referenced_packages
 import 'package:dio/dio.dart';
 
 class AppDio {

lib/data/network/backup_currency_api.dart

@@ -1,11 +1,12 @@
+// ignore: depend_on_referenced_packages
 import 'package:dio/dio.dart';
 import 'package:json_annotation/json_annotation.dart';
 import 'package:retrofit/retrofit.dart';
 
-import '../model/Currency.dart';
+import '../model/currency.dart';
 import 'app_dio.dart';
 
-part 'backupCurrencyApi.g.dart';
+part 'backup_currency_api.g.dart';
 
 @RestApi(baseUrl: "https://api.frankfurter.app/")
 abstract class BackupCurrencyApi {

lib/data/network/currency_api.dart

@@ -1,15 +1,16 @@
+// ignore: depend_on_referenced_packages
 import 'package:dio/dio.dart';
 import 'package:easy_cc_flutter/data/network/app_dio.dart';
 import 'package:json_annotation/json_annotation.dart';
 import 'package:retrofit/retrofit.dart';
 
-import '../model/Currency.dart';
+import '../model/currency.dart';
 
-part 'currencyApi.g.dart';
+part 'currency_api.g.dart';
 
 @RestApi(baseUrl: "https://free.currencyconverterapi.com/api/v3/")
 abstract class CurrencyApi {
-  factory CurrencyApi(Dio dio) = _CurrencyApi;
+  factory CurrencyApi(Dio dio, {String baseUrl}) = _CurrencyApi;
   static const api = String.fromEnvironment('currencyApiKey');
   
   static CurrencyApi create() {

lib/data/network/safe_api_call.dart

@@ -1,5 +1,6 @@
 import 'dart:io';
 
+// ignore: depend_on_referenced_packages
 import 'package:dio/dio.dart';
 import 'package:retrofit/retrofit.dart';
 

lib/data/prefs/preference_provider.dart

@@ -1,9 +1,9 @@
 import 'package:shared_preferences/shared_preferences.dart';
 
-import 'CurrencyPair.dart';
+import 'currency_pair.dart';
 
-const String CURRENCY_ONE = "conversion_one";
+const String currencyOne = "conversion_one";
-const String CURRENCY_TWO = "conversion_two";
+const String currencyTwo = "conversion_two";
 class PreferenceProvider {
   late final SharedPreferences _prefs;
 
@@ -12,13 +12,13 @@ class PreferenceProvider {
   }
 
   Future<void> saveConversionPair(String s1, String s2) async {
-    await _prefs.setString(CURRENCY_ONE, s1);
+    await _prefs.setString(currencyOne, s1);
-    await _prefs.setString(CURRENCY_TWO, s2);
+    await _prefs.setString(currencyTwo, s2);
   }
 
   CurrencyPair getConversionPair() {
-    String? s1 = _prefs.getString(CURRENCY_ONE);
+    String? s1 = _prefs.getString(currencyOne);
-    String? s2 = _prefs.getString(CURRENCY_TWO);
+    String? s2 = _prefs.getString(currencyTwo);
 
     return CurrencyPair(s1, s2);
   }

lib/data/repository/repository.dart

@@ -1,5 +1,5 @@
-import '../model/Currency.dart';
+import '../model/currency.dart';
-import '../prefs/CurrencyPair.dart';
+import '../prefs/currency_pair.dart';
 
 abstract class Repository {
   CurrencyPair getConversionPair();

lib/data/repository/repository_impl.dart

@@ -1,16 +1,16 @@
 import 'dart:io';
 
-import 'package:easy_cc_flutter/Utils/currencyUtils.dart';
+import 'package:easy_cc_flutter/Utils/currency_utils.dart';
-import 'package:easy_cc_flutter/data/model/Currency.dart';
+import 'package:easy_cc_flutter/data/model/currency.dart';
-import 'package:easy_cc_flutter/data/prefs/CurrencyPair.dart';
+import 'package:easy_cc_flutter/data/prefs/currency_pair.dart';
-import 'package:easy_cc_flutter/data/prefs/PreferenceProvider.dart';
 
 import '../../locator.dart';
 import '../../main.dart';
-import '../network/backupCurrencyApi.dart';
+import '../network/backup_currency_api.dart';
-import '../network/currencyApi.dart';
+import '../network/currency_api.dart';
-import '../network/safeApiCall.dart';
+import '../network/safe_api_call.dart';
-import 'Repository.dart';
+import '../prefs/preference_provider.dart';
+import 'repository.dart';
 
 class RepositoryImpl extends Repository with SafeApiCall {
   final PreferenceProvider _prefs = locator<PreferenceProvider>();

lib/home.dart

@@ -1,14 +1,16 @@
-import 'package:easy_cc_flutter/MainViewModel.dart';
+import 'package:easy_cc_flutter/main_view_model.dart';
-import 'package:easy_cc_flutter/Utils/SelectionType.dart';
+import 'package:easy_cc_flutter/Utils/selection_type.dart';
-import 'package:easy_cc_flutter/views/DropDownBox.dart';
+import 'package:easy_cc_flutter/views/drop_down_box.dart';
-import 'package:easy_cc_flutter/views/EditText.dart';
+import 'package:easy_cc_flutter/views/converter_edit_text.dart';
 import 'package:flutter/cupertino.dart';
 import 'package:flutter/material.dart';
 
-import 'BaseStatelessWidget.dart';
+import 'base_widget.dart';
-import 'Utils/Constants.dart';
+import 'Utils/constants.dart';
 
 class HomePage extends BaseStatelessWidget<MainViewModel> {
+  const HomePage({super.key});
+
   @override
   MainViewModel createViewModel() {
     return MainViewModel();

lib/locator.dart

@@ -1,11 +1,10 @@
-import 'package:dio/dio.dart';
+import 'package:easy_cc_flutter/main_view_model.dart';
-import 'package:easy_cc_flutter/MainViewModel.dart';
+import 'package:easy_cc_flutter/data/network/backup_currency_api.dart';
-import 'package:easy_cc_flutter/data/network/backupCurrencyApi.dart';
+import 'package:easy_cc_flutter/data/network/currency_api.dart';
-import 'package:easy_cc_flutter/data/network/currencyApi.dart';
+import 'package:easy_cc_flutter/data/repository/repository_impl.dart';
-import 'package:easy_cc_flutter/data/repository/RepositoryImpl.dart';
 import 'package:get_it/get_it.dart';
 
-import 'data/prefs/PreferenceProvider.dart';
+import 'data/prefs/preference_provider.dart';
 
 GetIt locator = GetIt.instance;
 

lib/main.dart

@@ -1,8 +1,8 @@
 import 'package:flutter/material.dart';
 import 'package:logger/logger.dart';
 
-import 'Home.dart';
+import 'data/prefs/preference_provider.dart';
-import 'data/prefs/PreferenceProvider.dart';
+import 'home.dart';
 import 'locator.dart';
 
 var logger = Logger(
@@ -36,7 +36,7 @@ class MyApp extends StatelessWidget {
         // is not restarted.
         primarySwatch: Colors.blue,
       ),
-      home: HomePage(),
+      home: const HomePage(),
     );
   }
 }

lib/main_view_model.dart

@@ -1,10 +1,10 @@
-import 'package:easy_cc_flutter/BaseViewModel.dart';
+import 'package:easy_cc_flutter/base_viewmodel.dart';
-import 'package:easy_cc_flutter/Utils/SelectionType.dart';
+import 'package:easy_cc_flutter/Utils/selection_type.dart';
 
-import 'Utils/Constants.dart';
+import 'Utils/constants.dart';
-import 'data/prefs/CurrencyPair.dart';
+import 'data/prefs/currency_pair.dart';
-import 'data/repository/Repository.dart';
+import 'data/repository/repository.dart';
-import 'data/repository/RepositoryImpl.dart';
+import 'data/repository/repository_impl.dart';
 import 'locator.dart';
 
 class MainViewModel extends BaseViewmodel {

lib/views/converter_edit_text.dart

@@ -1,7 +1,7 @@
 import 'package:flutter/material.dart';
 import 'package:flutter/services.dart';
 
-import '../Utils/Constants.dart';
+import '../Utils/constants.dart';
 
 class ConverterEditText extends StatelessWidget {
   final String _hintText;

lib/views/drop_down_box.dart

@@ -1,7 +1,7 @@
 import 'package:dropdown_search/dropdown_search.dart';
 import 'package:flutter/material.dart';
 
-import '../Utils/Constants.dart';
+import '../Utils/constants.dart';
 
 class DropDownBox extends StatelessWidget {
   final List<String> _selection;

test/unit_test/repository_test.dart

@@ -1,12 +1,13 @@
 import 'dart:io';
 
+// ignore: depend_on_referenced_packages
 import 'package:dio/dio.dart';
-import 'package:easy_cc_flutter/data/model/Currency.dart';
+import 'package:easy_cc_flutter/data/model/currency.dart';
-import 'package:easy_cc_flutter/data/network/backupCurrencyApi.dart';
+import 'package:easy_cc_flutter/data/network/backup_currency_api.dart';
-import 'package:easy_cc_flutter/data/network/currencyApi.dart';
+import 'package:easy_cc_flutter/data/network/currency_api.dart';
-import 'package:easy_cc_flutter/data/prefs/CurrencyPair.dart';
+import 'package:easy_cc_flutter/data/prefs/currency_pair.dart';
-import 'package:easy_cc_flutter/data/prefs/PreferenceProvider.dart';
+import 'package:easy_cc_flutter/data/prefs/preference_provider.dart';
-import 'package:easy_cc_flutter/data/repository/RepositoryImpl.dart';
+import 'package:easy_cc_flutter/data/repository/repository_impl.dart';
 import 'package:easy_cc_flutter/locator.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mockito/annotations.dart';

test/unit_test/viewmodel_test.dart

@@ -1,15 +1,15 @@
 import 'dart:io';
 
-import 'package:easy_cc_flutter/MainViewModel.dart';
+import 'package:easy_cc_flutter/main_view_model.dart';
-import 'package:easy_cc_flutter/Utils/SelectionType.dart';
+import 'package:easy_cc_flutter/Utils/selection_type.dart';
-import 'package:easy_cc_flutter/data/model/Currency.dart';
+import 'package:easy_cc_flutter/data/model/currency.dart';
-import 'package:easy_cc_flutter/data/prefs/CurrencyPair.dart';
+import 'package:easy_cc_flutter/data/prefs/currency_pair.dart';
-import 'package:easy_cc_flutter/data/repository/RepositoryImpl.dart';
+import 'package:easy_cc_flutter/data/repository/repository_impl.dart';
 import 'package:easy_cc_flutter/locator.dart';
 import 'package:flutter_test/flutter_test.dart';
 import 'package:mockito/annotations.dart';
 import 'package:mockito/mockito.dart';
-import 'package:easy_cc_flutter/Utils/ViewState.dart';
+import 'package:easy_cc_flutter/Utils/view_state.dart';
 
 import 'viewmodel_test.mocks.dart';