danger-spm_version_updates 0.1.2 → 0.1.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6f3edb5f215f830ec9996754f188804ca79629de571e3f6d5d462c4e6a78f4ed
-  data.tar.gz: cc4e01f131130d1564fc0e5e65d64ff2f0ebe9beeae315c303117a932e805bd8
+  metadata.gz: 45434b8470d53708ee3261e6415b61eb6ca22dd4c3bd2080cbb0f28014c48276
+  data.tar.gz: c41a9d67f408fdaeaec72bdca65a8b0abefd998c0e852d9ce1f4c4ed76bda79c
 SHA512:
-  metadata.gz: 45ac718c4f0506e142277fcbe61a1fe6879bc0e0e49b760d240941076418cb0d7ccc491fa89e1e57b6bb203381e6557032ee45bf87c900821151ab4c356d5ccc
-  data.tar.gz: 40045fcf48475db57dec502b84934adfaea9c3a2d73794df73799d963d5d16d115e877fd54fe4cb8de3d9576e910f2ea93187709c0df11e9c0b50149843a8690
+  metadata.gz: 949714767a85ebbf99c10aa1e8b424344a54a0cd3d8c7e1dcaae759b3f656a92f45b6df6adf9bf90468448f80636025ec659758c9309bf1a3c723f2c24ccfe18
+  data.tar.gz: 75e265f2a49a1efddae3c0272818ef67e2254739ce100bf3551306e670fe0e7e9fc5b608a49e91aa7b218bde1a92ae2d856ee87bd8e58276d4d01d53e18932ee

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    danger-spm_version_updates (0.1.2)
+    danger-spm_version_updates (0.1.3)
       danger-plugin-api (~> 1.0)
       semantic (~> 1.6)
       xcodeproj (~> 1.24)

data/README.md

@@ -45,7 +45,7 @@ spm_version_updates.report_above_maximum = true
 # Whether to report pre-release versions, default false
 spm_version_updates.report_pre_releases = true
 
-# A list of repositories to ignore entirely, must exactly match the URL as configured in the Xcode project
+# A list of repository URLs for packages to ignore entirely
 spm_version_updates.ignore_repos = ["https://github.com/pointfreeco/swift-snapshot-testing"]
 ```
 

data/danger-spm_version_updates.gemspec

@@ -15,7 +15,7 @@ Gem::Specification.new do |spec|
   spec.license       = "MIT"
   spec.required_ruby_version = ">= 3.0"
 
-  spec.files         = `git ls-files`.split($/)
+  spec.files         = Dir['lib/*'] + Dir['lib/**/*'] + Dir['*']
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
   spec.metadata["rubygems_mfa_required"] = "true"

data/lib/spm_version_updates/gem_version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SpmVersionUpdates
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/lib/spm_version_updates/git.rb

@@ -0,0 +1,56 @@
+# frozen_string_literal: true
+
+module Git
+  # Removes protocol and trailing .git from a repo URL
+  # @param   [String] repo_url
+  #          The URL of the repository
+  # @return [String]
+  def self.trim_repo_url(repo_url)
+    repo_url.split("://").last.gsub(/\.git$/, "")
+  end
+
+  # Extract a readable name for the repo given the url, generally org/repo
+  # @return [String]
+  def self.repo_name(repo_url)
+    match = repo_url.match(%r{([\w-]+/[\w-]+)(.git)?$})
+
+    if match
+      match[1] || match[0]
+    else
+      repo_url
+    end
+  end
+
+  # Call git to list tags
+  # @param   [String] repo_url
+  #          The URL of the dependency's repository
+  # @return [Array<Semantic::Version>]
+  def self.version_tags(repo_url)
+    versions = `git ls-remote -t #{repo_url}`
+      .split("\n")
+      .map { |line| line.split("/tags/").last }
+      .filter_map { |line|
+        begin
+          Semantic::Version.new(line)
+        rescue ArgumentError
+          nil
+        end
+      }
+    versions.sort!
+    versions.reverse!
+    versions
+  end
+
+  # Call git to find the last commit on a branch
+  # @param   [String] repo_url
+  #          The URL of the dependency's repository
+  # @param   [String] branch_name
+  #          The name of the branch on which to find the last commit
+  # @return [String]
+  def self.branch_last_commit(repo_url, branch_name)
+    `git ls-remote -h #{repo_url}`
+      .split("\n")
+      .find { |line| line.split("\trefs/heads/")[1] == branch_name }
+      .split("\trefs/heads/")[0]
+  end
+end

data/lib/spm_version_updates/plugin.rb

@@ -1,7 +1,8 @@
 # frozen_string_literal: true
 
 require "semantic"
-require "xcodeproj"
+require_relative "git"
+require_relative "xcode"
 
 module Danger
   # A Danger plugin for checking if there are versions upgrades available for SPM dependencies
@@ -25,38 +26,42 @@ module Danger
     # @return   [Boolean]
     attr_accessor :report_pre_releases
 
-    # A list of repositories to ignore entirely, must exactly match the URL as configured in the Xcode project
+    # A list of repository URLs for packages to ignore entirely
     # @return   [Array<String>]
     attr_accessor :ignore_repos
 
     # A method that you can call from your Dangerfile
     # @param   [String] xcodeproj_path
     #          The path to your Xcode project
+    # @raise [XcodeprojPathMustBeSet] if the xcodeproj_path is blank
     # @return   [void]
     def check_for_updates(xcodeproj_path)
-      remote_packages = get_remote_package(xcodeproj_path)
-      resolved_versions = get_resolved_versions(xcodeproj_path)
+      remote_packages = Xcode.get_packages(xcodeproj_path)
+      resolved_versions = Xcode.get_resolved_versions(xcodeproj_path)
+      $stderr.puts("Found resolved versions for #{resolved_versions.size} packages")
+
+      self.ignore_repos = self.ignore_repos&.map! { |repo| Git.trim_repo_url(repo) }
 
       remote_packages.each { |repository_url, requirement|
-        next if ignore_repos&.include?(repository_url)
+        next if self.ignore_repos&.include?(repository_url)
 
-        name = repo_name(repository_url)
+        name = Git.repo_name(repository_url)
         resolved_version = resolved_versions[repository_url]
         kind = requirement["kind"]
 
         if resolved_version.nil?
-          $stderr.puts("Unable to locate the current version for #{name} (#{repository_url}) in #{find_packages_resolved_file(xcodeproj_path)}")
+          $stderr.puts("Unable to locate the current version for #{name} (#{repository_url})")
           next
         end
 
         if kind == "branch"
           branch = requirement["branch"]
-          last_commit = git_branch_last_commit(repository_url, branch)
+          last_commit = Git.branch_last_commit(repository_url, branch)
           warn("Newer commit available for #{name} (#{branch}): #{last_commit}") unless last_commit == resolved_version
           next
         end
 
-        available_versions = git_versions(repository_url)
+        available_versions = Git.version_tags(repository_url)
         next if available_versions.first.to_s == resolved_version
 
         if kind == "exactVersion" && @check_when_exact
@@ -71,60 +76,6 @@ module Danger
       }
     end
 
-    # Extracts remote packages from an Xcode project
-    # @param   [String] xcodeproj_path
-    #          The path to your Xcode project
-    # @return [Hash<String, Hash>]
-    def get_remote_package(xcodeproj_path)
-      raise(XcodeprojPathMustBeSet) if xcodeproj_path.nil?
-
-      filter_remote_packages(Xcodeproj::Project.open(xcodeproj_path))
-    end
-
-    # Extracts resolved versions from Package.resolved relative to an Xcode project
-    # @param   [String] xcodeproj_path
-    #          The path to your Xcode project
-    # @return [Hash<String, String>]
-    def get_resolved_versions(xcodeproj_path)
-      resolved_path = find_packages_resolved_file(xcodeproj_path)
-      raise(CouldNotFindResolvedFile) unless File.exist?(resolved_path)
-
-      JSON.load_file!(resolved_path)["pins"]
-        .to_h { |pin| [pin["location"], pin["state"]["version"] || pin["state"]["revision"]] }
-    end
-
-    # Extract a readable name for the repo given the url, generally org/repo
-    # @return [String]
-    def repo_name(repo_url)
-      match = repo_url.match(%r{([\w-]+/[\w-]+)(.git)?$})
-
-      if match
-        match[1] || match[0]
-      else
-        repo_url
-      end
-    end
-
-    # Find the configured SPM dependencies in the xcodeproj
-    # @return [Hash<String, Hash>]
-    def filter_remote_packages(project)
-      project.objects.select { |obj|
-        obj.kind_of?(Xcodeproj::Project::Object::XCRemoteSwiftPackageReference) &&
-          obj.requirement["kind"] != "commit"
-      }
-        .to_h { |package| [package.repositoryURL, package.requirement] }
-    end
-
-    # Find the Packages.resolved file
-    # @return [String]
-    def find_packages_resolved_file(xcodeproj_path)
-      if Dir.exist?(xcodeproj_path.sub("xcodeproj", "xcworkspace"))
-        File.join(xcodeproj_path.sub("xcodeproj", "xcworkspace"), "xcshareddata", "swiftpm", "Package.resolved")
-      else
-        File.join(xcodeproj_path, "project.xcworkspace", "xcshareddata", "swiftpm", "Package.resolved")
-      end
-    end
-
     private
 
     def warn_for_new_versions_exact(available_versions, name, resolved_version)
@@ -173,44 +124,5 @@ Newest version of #{name}: #{available_versions.first} (but this package is conf
         TEXT
       ) unless newest_above_reqs == newest_meeting_reqs || newest_meeting_reqs.to_s == resolved_version
     end
-
-    # Call git to list tags
-    # @param   [String] repo_url
-    #          The URL of the dependency's repository
-    # @return [Array<Semantic::Version>]
-    def git_versions(repo_url)
-      versions = `git ls-remote -t #{repo_url}`
-        .split("\n")
-        .map { |line| line.split("/tags/").last }
-        .filter_map { |line|
-          begin
-            Semantic::Version.new(line)
-          rescue ArgumentError
-            nil
-          end
-        }
-      versions.sort!
-      versions.reverse!
-      versions
-    end
-
-    # Calkl git to find the last commit on a branch
-    # @param   [String] repo_url
-    #          The URL of the dependency's repository
-    # @param   [String] branch_name
-    #          The name of the branch on which to find the last commit
-    # @return [String]
-    def git_branch_last_commit(repo_url, branch_name)
-      `git ls-remote -h #{repo_url}`
-        .split("\n")
-        .find { |line| line.split("\trefs/heads/")[1] == branch_name }
-        .split("\trefs/heads/")[0]
-    end
-  end
-
-  class XcodeprojPathMustBeSet < StandardError
-  end
-
-  class CouldNotFindResolvedFile < StandardError
   end
 end

data/lib/spm_version_updates/xcode.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require "xcodeproj"
+
+module Xcode
+  # Find the configured SPM dependencies in the xcodeproj
+  # @param   [String] xcodeproj_path
+  #          The path of the Xcode project
+  # @return [Hash<String, Hash>]
+  def self.get_packages(xcodeproj_path)
+    raise(XcodeprojPathMustBeSet) if xcodeproj_path.nil? || xcodeproj_path.empty?
+
+    project = Xcodeproj::Project.open(xcodeproj_path)
+    project.objects.select { |obj|
+      obj.kind_of?(Xcodeproj::Project::Object::XCRemoteSwiftPackageReference) &&
+        obj.requirement["kind"] != "commit"
+    }
+      .to_h { |package|
+        [Git.trim_repo_url(package.repositoryURL), package.requirement]
+      }
+  end
+
+  # Extracts resolved versions from Package.resolved relative to an Xcode project
+  # @param   [String] xcodeproj_path
+  #          The path to your Xcode project
+  # @raise [CouldNotFindResolvedFile] if no Package.resolved files were found
+  # @return [Hash<String, String>]
+  def self.get_resolved_versions(xcodeproj_path)
+    resolved_paths = find_packages_resolved_file(xcodeproj_path)
+    raise(CouldNotFindResolvedFile) if resolved_paths.empty?
+
+    resolved_versions = resolved_paths.map { |resolved_path|
+      JSON.load_file!(resolved_path)["pins"]
+        .to_h { |pin|
+          [Git.trim_repo_url(pin["location"]), pin["state"]["version"] || pin["state"]["revision"]]
+        }
+    }
+    resolved_versions.reduce(:merge!)
+  end
+
+  # Find the Packages.resolved file
+  # @return [Array<String>]
+  def self.find_packages_resolved_file(xcodeproj_path)
+    locations = []
+    # First check the workspace for a resolved file
+    workspace = xcodeproj_path.sub("xcodeproj", "xcworkspace")
+    if Dir.exist?(workspace)
+      path = File.join(workspace, "xcshareddata", "swiftpm", "Package.resolved")
+      locations << path if File.exist?(path)
+    end
+
+    # Then check the project for a resolved file
+    path = File.join(xcodeproj_path, "project.xcworkspace", "xcshareddata", "swiftpm", "Package.resolved")
+    locations << path if File.exist?(path)
+
+    $stderr.puts("Searching for resolved packages in: #{locations}")
+    locations
+  end
+
+  private_class_method :find_packages_resolved_file
+
+  class XcodeprojPathMustBeSet < StandardError
+  end
+
+  class CouldNotFindResolvedFile < StandardError
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-spm_version_updates
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Harold Martin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2024-03-18 00:00:00.000000000 Z
+date: 2024-03-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: danger-plugin-api
@@ -284,20 +284,6 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".editorconfig"
-- ".github/workflows/docs.yml"
-- ".github/workflows/lint_and_test.yml"
-- ".github/workflows/push_gem.yml"
-- ".github/workflows/readme.yml"
-- ".gitignore"
-- ".idea/.gitignore"
-- ".idea/danger-plugin-spm-version-updates.iml"
-- ".idea/misc.xml"
-- ".idea/modules.xml"
-- ".idea/vcs.xml"
-- ".markdownlint.jsonc"
-- ".reek.yml"
-- ".rubocop.yml"
 - CODE_OF_CONDUCT.md
 - Gemfile
 - Gemfile.lock
@@ -309,20 +295,9 @@ files:
 - lib/danger_plugin.rb
 - lib/danger_spm_version_updates.rb
 - lib/spm_version_updates/gem_version.rb
+- lib/spm_version_updates/git.rb
 - lib/spm_version_updates/plugin.rb
-- spec/spec_helper.rb
-- spec/spm_version_updates_spec.rb
-- spec/support/fixtures/Branch.xcodeproj/project.pbxproj
-- spec/support/fixtures/Branch.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
-- spec/support/fixtures/ExactVersion.xcodeproj/project.pbxproj
-- spec/support/fixtures/ExactVersion.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
-- spec/support/fixtures/NoResolvedVersion.xcodeproj/project.pbxproj
-- spec/support/fixtures/NoResolvedVersion.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
-- spec/support/fixtures/UpToNextMajor.xcodeproj/project.pbxproj
-- spec/support/fixtures/UpToNextMajor.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
-- spec/support/fixtures/VersionRange.xcodeproj/project.pbxproj
-- spec/support/fixtures/VersionRange.xcodeproj/project.xcworkspace/xcshareddata/swiftpm/Package.resolved
-- spec/support/fixtures/github_pr.json
+- lib/spm_version_updates/xcode.rb
 homepage: https://github.com/hbmartin/danger-spm_version_updates
 licenses:
 - MIT

data/.editorconfig

@@ -1,15 +0,0 @@
-root = true
-
-[*]
-end_of_line = lf
-insert_final_newline = true
-charset = utf-8
-trim_trailing_whitespace = true
-
-[*.{rb,yml,gemspec}]
-indent_style = space
-indent_size = 2
-
-[Dangerfile, Rakefile, Guardfile, Gemfile]
-indent_style = space
-indent_size = 2

data/.github/workflows/docs.yml

@@ -1,43 +0,0 @@
-name: Publish Yard HTML to GitHub Pages
-
-on:
-  push:
-    tags:
-      - v*
-  workflow_dispatch:
-
-jobs:
-  build:
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v3
-      - name: Set up Ruby
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: 3.2
-          bundler-cache: true
-      - name: Build HTML docs
-        run: bundle exec yard
-      - name: Upload artifact
-        uses: actions/upload-pages-artifact@v2
-        with:
-          path: ./doc
-  deploy:
-    needs: build
-
-    # Grant GITHUB_TOKEN the permissions required to make a Pages deployment
-    permissions:
-      pages: write # to deploy to Pages
-      id-token: write # to verify the deployment originates from an appropriate source
-
-    # Deploy to the github-pages environment
-    environment:
-      name: github-pages
-      url: ${{ steps.deployment.outputs.page_url }}
-
-    # Specify runner + deployment step
-    runs-on: ubuntu-latest
-    steps:
-      - name: Deploy to GitHub Pages
-        id: deployment
-        uses: actions/deploy-pages@v2 # or the latest "vX.X.X" version tag for this action

data/.github/workflows/lint_and_test.yml

@@ -1,32 +0,0 @@
-name: CI
-
-on:
-  push:
-    branches: [ main ]
-  pull_request:
-    branches: [ main ]
-
-jobs:
-  build:
-    strategy:
-      matrix:
-        ruby-version:
-          - "3.0"
-          - "3.1"
-          - "3.2"
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v4
-      - name: Set up Ruby
-        uses: ruby/setup-ruby@v1
-        with:
-          ruby-version: ${{ matrix.ruby-version }}
-          bundler-cache: true
-      - name: Running Tests
-        run: bundle exec rake spec
-      - name: Upload coverage reports to Codecov
-        uses: codecov/codecov-action@v4
-        with:
-          fail_ci_if_error: true
-        env:
-          CODECOV_TOKEN: ${{ secrets.CODECOV_TOKEN }}

data/.github/workflows/push_gem.yml

@@ -1,26 +0,0 @@
-name: Push Gem
-
-on:
-  push:
-    tags:
-      - v*
-
-jobs:
-  push:
-    runs-on: ubuntu-latest
-
-    permissions:
-      contents: write
-      id-token: write
-
-    steps:
-      # Set up
-      - uses: actions/checkout@v4
-      - name: Set up Ruby
-        uses: ruby/setup-ruby@v1
-        with:
-          bundler-cache: true
-          ruby-version: ruby
-
-      # Release
-      - uses: rubygems/release-gem@v1

data/.github/workflows/readme.yml

@@ -1,21 +0,0 @@
-name: Readme formatting and typo checking
-
-on:
-  push:
-    branches: ["main"]
-  pull_request:
-    branches: ["main"]
-
-jobs:
-  lint:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Check out repository code
-        uses: actions/checkout@v4
-      - name: Markdown Lint
-        uses: avto-dev/markdown-lint@v1.5.0
-        with:
-          args: "./README.md"
-          config: "./.markdownlint.jsonc"
-      - name: Typos
-        uses: crate-ci/typos@v1.19.0

data/.gitignore

@@ -1,136 +0,0 @@
-.DS_Store
-
-*.gem
-*.rbc
-/.config
-/coverage/
-/InstalledFiles
-/pkg/
-/spec/reports/
-/spec/examples.txt
-/test/tmp/
-/test/version_tmp/
-/tmp/
-
-# Used by dotenv library to load environment variables.
-# .env
-
-# Ignore Byebug command history file.
-.byebug_history
-
-## Specific to RubyMotion:
-.dat*
-.repl_history
-build/
-*.bridgesupport
-build-iPhoneOS/
-build-iPhoneSimulator/
-
-## Specific to RubyMotion (use of CocoaPods):
-#
-# We recommend against adding the Pods directory to your .gitignore. However
-# you should judge for yourself, the pros and cons are mentioned at:
-# https://guides.cocoapods.org/using/using-cocoapods.html#should-i-check-the-pods-directory-into-source-control
-#
-# vendor/Pods/
-
-## Documentation cache and generated files:
-/.yardoc/
-/_yardoc/
-/doc/
-/rdoc/
-
-## Environment normalization:
-/.bundle/
-/vendor/bundle
-/lib/bundler/man/
-
-# for a library or gem, you might want to ignore these files since the code is
-# intended to run in multiple environments; otherwise, check them in:
-# Gemfile.lock
-# .ruby-version
-# .ruby-gemset
-
-# unless supporting rvm < 1.11.0 or doing something fancy, ignore this:
-.rvmrc
-
-# Used by RuboCop. Remote config files pulled in from inherit_from directive.
- .rubocop-https?--*
-
-# Covers JetBrains IDEs: IntelliJ, RubyMine, PhpStorm, AppCode, PyCharm, CLion, Android Studio, WebStorm and Rider
-# Reference: https://intellij-support.jetbrains.com/hc/en-us/articles/206544839
-
-# User-specific stuff
-.idea/**/workspace.xml
-.idea/**/tasks.xml
-.idea/**/usage.statistics.xml
-.idea/**/dictionaries
-.idea/**/shelf
-
-# AWS User-specific
-.idea/**/aws.xml
-
-# Generated files
-.idea/**/contentModel.xml
-
-# Sensitive or high-churn files
-.idea/**/dataSources/
-.idea/**/dataSources.ids
-.idea/**/dataSources.local.xml
-.idea/**/sqlDataSources.xml
-.idea/**/dynamic.xml
-.idea/**/uiDesigner.xml
-.idea/**/dbnavigator.xml
-
-# Gradle
-.idea/**/gradle.xml
-.idea/**/libraries
-
-# Gradle and Maven with auto-import
-# When using Gradle or Maven with auto-import, you should exclude module files,
-# since they will be recreated, and may cause churn.  Uncomment if using
-# auto-import.
-# .idea/artifacts
-# .idea/compiler.xml
-# .idea/jarRepositories.xml
-# .idea/modules.xml
-# .idea/*.iml
-# .idea/modules
-# *.iml
-# *.ipr
-
-# CMake
-cmake-build-*/
-
-# Mongo Explorer plugin
-.idea/**/mongoSettings.xml
-
-# File-based project format
-*.iws
-
-# IntelliJ
-out/
-
-# mpeltonen/sbt-idea plugin
-.idea_modules/
-
-# JIRA plugin
-atlassian-ide-plugin.xml
-
-# Cursive Clojure plugin
-.idea/replstate.xml
-
-# SonarLint plugin
-.idea/sonarlint/
-
-# Crashlytics plugin (for Android Studio and IntelliJ)
-com_crashlytics_export_strings.xml
-crashlytics.properties
-crashlytics-build.properties
-fabric.properties
-
-# Editor-based Rest Client
-.idea/httpRequests
-
-# Android studio 3.1+ serialized cache file
-.idea/caches/build_file_checksums.ser

data/.idea/.gitignore

@@ -1,8 +0,0 @@
-# Default ignored files
-/shelf/
-/workspace.xml
-# Editor-based HTTP Client requests
-/httpRequests/
-# Datasource local storage ignored files
-/dataSources/
-/dataSources.local.xml