From 7bcda64781041ad005c08661fa6970b091da7070 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Fabi=C3=A1n=20Cancela?=
 <62720589+fcancela@users.noreply.github.com>
Date: Mon, 9 May 2022 15:10:37 -0300
Subject: [PATCH] Hotfix/remove unused dependencies (#19)

* Remove unnecesary dependencies and update readme

* Change release version to 0.2.2

* remove-dependencies: Add Changelog entry
---
 CHANGELOG.md         |  10 ++++
 README.md            |  14 ++++-
 example/pubspec.lock | 123 +------------------------------------------
 pubspec.lock         |   4 +-
 pubspec.yaml         |   6 +--
 5 files changed, 30 insertions(+), 127 deletions(-)

diff --git a/CHANGELOG.md b/CHANGELOG.md
index e96aa14..ae15b3d 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -1,3 +1,13 @@
+## 0.2.2
+
+### Changed
+
+- Added information needed in order to properly run the README.md Getting Started example app.
+
+### Fixed
+
+- Moved Mockito and dotenv depenencies as dev dependencies
+
 ## 0.2.1
 
 This is the first official release of the Millicast Flutter SDK.
diff --git a/README.md b/README.md
index 046be4a..8d18ef7 100644
--- a/README.md
+++ b/README.md
@@ -140,6 +140,9 @@ class _MyHomePageState extends State<MyHomePage> {
   @override
   void initState() {
     initRenderers();
+    // Run application with --dart-define type flag like:
+    // flutter run --dart-define=type="subscribe||publish"
+    // To choose wether you want to publishe or subscribe.
     switch (type) {
       case 'subscribe':
         subscribeExample();
@@ -148,6 +151,7 @@ class _MyHomePageState extends State<MyHomePage> {
         publishExample();
         break;
       default:
+        publishExample();
     }
     super.initState();
   }
@@ -269,6 +273,14 @@ Future viewConnect(RTCVideoRenderer localRenderer) async {
 }
 ```
 
+### Run the application
+
+Set `type` environment variable `publish` or `subscribe` to decide wether you want to run the publisher or viewer app.
+
+```bash
+flutter run --dart-define=type='publish'
+```
+
 ### Important reminder
 
 When you compile the release apk, you need to add the following operations:
@@ -276,7 +288,7 @@ When you compile the release apk, you need to add the following operations:
 
 ## API Reference
 
-You can find the latest, most up to date SDK documentation at our [API Reference page](https://millicast.github.io/flutter-sdk/). There are more examples with every module available.
+You can find the latest, most up to date SDK documentation at our [API Reference page](https://pub.dev/documentation/millicast_flutter_sdk/latest/). There are more examples with every module available.
 
 ## Sample
 
diff --git a/example/pubspec.lock b/example/pubspec.lock
index f89e50a..918c7c1 100644
--- a/example/pubspec.lock
+++ b/example/pubspec.lock
@@ -1,20 +1,6 @@
 # Generated by pub
 # See https://dart.dev/tools/pub/glossary#lockfile
 packages:
-  _fe_analyzer_shared:
-    dependency: transitive
-    description:
-      name: _fe_analyzer_shared
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "38.0.0"
-  analyzer:
-    dependency: transitive
-    description:
-      name: analyzer
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "3.4.1"
   archive:
     dependency: transitive
     description:
@@ -43,27 +29,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "2.1.0"
-  build:
-    dependency: transitive
-    description:
-      name: build
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "2.2.1"
-  built_collection:
-    dependency: transitive
-    description:
-      name: built_collection
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "5.1.1"
-  built_value:
-    dependency: transitive
-    description:
-      name: built_value
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "8.1.4"
   characters:
     dependency: transitive
     description:
@@ -85,13 +50,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.1.0"
-  code_builder:
-    dependency: transitive
-    description:
-      name: code_builder
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "4.1.0"
   collection:
     dependency: transitive
     description:
@@ -99,13 +57,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.15.0"
-  convert:
-    dependency: transitive
-    description:
-      name: convert
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "3.0.1"
   crypto:
     dependency: transitive
     description:
@@ -120,13 +71,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.0.4"
-  dart_style:
-    dependency: transitive
-    description:
-      name: dart_style
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "2.2.2"
   dart_webrtc:
     dependency: transitive
     description:
@@ -134,13 +78,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.0.5"
-  dotenv:
-    dependency: transitive
-    description:
-      name: dotenv
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "4.0.1"
   eventify:
     dependency: transitive
     description:
@@ -169,13 +106,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "6.1.2"
-  fixnum:
-    dependency: transitive
-    description:
-      name: fixnum
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "1.0.0"
   flutter:
     dependency: "direct main"
     description: flutter
@@ -226,13 +156,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "0.8.5"
-  glob:
-    dependency: transitive
-    description:
-      name: glob
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "2.0.2"
   http:
     dependency: "direct main"
     description:
@@ -282,13 +205,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.1.0"
-  logging:
-    dependency: transitive
-    description:
-      name: logging
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "1.0.2"
   matcher:
     dependency: transitive
     description:
@@ -316,7 +232,7 @@ packages:
       path: ".."
       relative: true
     source: path
-    version: "0.2.1"
+    version: "0.2.2"
   mime:
     dependency: transitive
     description:
@@ -324,20 +240,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "1.0.1"
-  mockito:
-    dependency: transitive
-    description:
-      name: mockito
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "5.1.0"
-  package_config:
-    dependency: transitive
-    description:
-      name: package_config
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "2.0.2"
   path:
     dependency: transitive
     description:
@@ -422,13 +324,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "4.2.4"
-  pub_semver:
-    dependency: transitive
-    description:
-      name: pub_semver
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "2.1.1"
   rxdart:
     dependency: transitive
     description:
@@ -490,13 +385,6 @@ packages:
     description: flutter
     source: sdk
     version: "0.0.99"
-  source_gen:
-    dependency: transitive
-    description:
-      name: source_gen
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "1.2.1"
   source_span:
     dependency: transitive
     description:
@@ -637,13 +525,6 @@ packages:
       url: "https://pub.dartlang.org"
     source: hosted
     version: "2.1.1"
-  watcher:
-    dependency: transitive
-    description:
-      name: watcher
-      url: "https://pub.dartlang.org"
-    source: hosted
-    version: "1.0.1"
   web_socket_channel:
     dependency: "direct main"
     description:
@@ -687,5 +568,5 @@ packages:
     source: hosted
     version: "3.1.0"
 sdks:
-  dart: ">=2.16.0 <3.0.0"
+  dart: ">=2.15.1 <3.0.0"
   flutter: ">=2.10.0"
diff --git a/pubspec.lock b/pubspec.lock
index 06b7627..904aaff 100644
--- a/pubspec.lock
+++ b/pubspec.lock
@@ -191,7 +191,7 @@ packages:
     source: hosted
     version: "4.1.0"
   dotenv:
-    dependency: "direct main"
+    dependency: "direct dev"
     description:
       name: dotenv
       url: "https://pub.dartlang.org"
@@ -390,7 +390,7 @@ packages:
     source: hosted
     version: "1.0.2"
   mockito:
-    dependency: "direct main"
+    dependency: "direct dev"
     description:
       name: mockito
       url: "https://pub.dartlang.org"
diff --git a/pubspec.yaml b/pubspec.yaml
index b46bd0b..05565dd 100644
--- a/pubspec.yaml
+++ b/pubspec.yaml
@@ -1,6 +1,6 @@
 name: millicast_flutter_sdk
 description: A Flutter SDK that allows developers to simplify Millicast services integration into their own Android and iOS apps.
-version: 0.2.1
+version: 0.2.2
 homepage: https://github.com/millicast/flutter-sdk
 
 environment:
@@ -9,13 +9,11 @@ environment:
 
 dependencies:
   eventify: ^1.0.0
-  dotenv: ^4.0.0
   flutter:
     sdk: flutter
   logger: ^1.1.0
   flutter_webrtc: ^0.8.1
   http: ^0.13.4
-  mockito: ^5.0.17
   web_socket_channel: ^2.1.0
   jwt_decode: ^0.3.1
   sdp_transform: ^0.3.2
@@ -27,6 +25,8 @@ dev_dependencies:
   dartdoc: ^4.1.0
   test: ^1.17.12
   build_runner: ^2.1.7
+  mockito: ^5.0.17
+  dotenv: ^4.0.0
 
 # For information on the generic Dart part of this file, see the
 # following page: https://dart.dev/tools/pub/pubspec