Merge pull request #7602 from amorde/cleanup-todos

Cleanup old TODOs

lib/cocoapods/command/outdated.rb

@@ -13,11 +13,6 @@ module Pod
 
       # Run the command
       #
-      # @todo the command reports new dependencies added to the Podfile as
-      #       updates.
-      #
-      # @todo fix.
-      #
       def run
         if updates.empty?
           UI.puts 'No pod updates are available.'.yellow

lib/cocoapods/installer.rb

@@ -544,8 +544,6 @@ module Pod
 
     # Writes the Podfile and the lock files.
     #
-    # @todo   Pass the checkout options to the Lockfile.
-    #
     # @return [void]
     #
     def write_lockfiles
@@ -571,11 +569,6 @@ module Pod
     #
     # @return [void]
     #
-    # @todo   [#397] The libraries should be cleaned and the re-added on every
-    #         installation. Maybe a clean_user_project phase should be added.
-    #         In any case it appears to be a good idea store target definition
-    #         information in the lockfile.
-    #
     def integrate_user_project
       UI.section "Integrating client #{'project'.pluralize(aggregate_targets.map(&:user_project_path).uniq.count)}" do
         installation_root = config.installation_root

lib/cocoapods/installer/analyzer.rb

@@ -223,9 +223,6 @@ module Pod
       #         the name of the Pod (root name of the dependencies) and doesn't
       #         group them by target definition.
       #
-      # @todo   [CocoaPods > 0.18] If there isn't a Lockfile all the Pods should
-      #         be marked as added.
-      #
       def generate_podfile_state
         if lockfile
           pods_state = nil
@@ -706,8 +703,6 @@ module Pod
       #
       # @return [void]
       #
-      # TODO:    Specs
-      #
       def fetch_external_sources
         return unless allow_pre_downloads?
 

lib/cocoapods/installer/analyzer/sandbox_analyzer.rb

@@ -43,11 +43,7 @@ module Pod
         alias_method :update_mode?, :update_mode
 
         # @return [Lockfile] The lockfile of the installation as a fall-back if
-        #         there is no sandbox manifest. This is indented as a temporary
-        #         solution to prevent the full re-installation from users which
-        #         are upgrading from CP < 0.17.
-        #
-        # @todo   Remove for CP 0.18.
+        #         there is no sandbox manifest.
         #
         attr_reader :lockfile
 

lib/cocoapods/installer/pod_source_installer.rb

@@ -63,9 +63,6 @@ module Pod
 
       # Cleans the installations if appropriate.
       #
-      # @todo   As the pre install hooks need to run before cleaning this
-      #         method should be refactored.
-      #
       # @return [void]
       #
       def clean!
@@ -74,9 +71,6 @@ module Pod
 
       # Locks the source files if appropriate.
       #
-      # @todo   As the pre install hooks need to run before cleaning this
-      #         method should be refactored.
-      #
       # @return [void]
       #
       def lock_files!(file_accessors)
@@ -88,9 +82,6 @@ module Pod
 
       # Unlocks the source files if appropriate.
       #
-      # @todo   As the pre install hooks need to run before cleaning this
-      #         method should be refactored.
-      #
       # @return [void]
       #
       def unlock_files!(file_accessors)

lib/cocoapods/installer/user_project_integrator.rb

@@ -44,8 +44,6 @@ module Pod
       # @param  [Pathname] installation_root @see #installation_root
       # @param  [Array<AggregateTarget>]  targets @see #targets
       #
-      # @todo   Too many initialization arguments
-      #
       def initialize(podfile, sandbox, installation_root, targets)
         @podfile = podfile
         @sandbox = sandbox

lib/cocoapods/sandbox.rb

@@ -367,8 +367,6 @@ module Pod
     # @return [Hash{String=>Pathname}] The path of the Pods' podspecs with a local source
     #         grouped by their root name.
     #
-    # @todo   Rename (e.g. `pods_with_local_path`)
-    #
     attr_reader :development_pods
 
     # Checks if a Pod is locally sourced?