From 1fbd35d26bbd443d7bf58e00e3709217810940af Mon Sep 17 00:00:00 2001 From: kshann Date: Fri, 17 Jan 2025 15:33:33 +1100 Subject: [PATCH 1/5] Add gems no longer bundled within ruby 3.4.x (#25) --- .github/workflows/test.yml | 1 + Gemfile | 5 +++++ 2 files changed, 6 insertions(+) diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index e2b8bb4..bc61863 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -1,6 +1,7 @@ name: Test on: + workflow_dispatch: {} push: jobs: diff --git a/Gemfile b/Gemfile index fc7fa4a..fa11f08 100644 --- a/Gemfile +++ b/Gemfile @@ -21,6 +21,11 @@ gem 'rubocop-require_tools' # SimpleCov is a code coverage analysis tool for Ruby. gem 'simplecov' +# Dependencies no longer included in newer Ruby versions +gem 'abbrev' +gem 'logger' +gem 'mutex_m' + gemspec plugins_path = File.join(File.dirname(__FILE__), 'fastlane', 'Pluginfile') From b9d8bd9bf41923f36442c1b06a8abb4ae790250d Mon Sep 17 00:00:00 2001 From: kshann Date: Mon, 20 Jan 2025 13:37:07 +1100 Subject: [PATCH 2/5] iOS release changes (#24) --- .../actions/asana_find_release_task_action.rb | 2 +- .../actions/tag_release_action.rb | 6 ++--- ...ernal-release-complete-with-tasks.html.erb | 4 +-- .../internal-release-complete.html.erb | 5 ++-- ...ternal-release-ready-merge-failed.html.erb | 2 +- ...internal-release-ready-tag-failed.html.erb | 2 +- .../templates/internal-release-ready.html.erb | 4 +-- .../helper/asana_helper.rb | 2 +- .../helper/ddg_apple_automation_helper.rb | 25 ++++++++++++++----- .../plugin/ddg_apple_automation/version.rb | 2 +- spec/asana_add_comment_action_spec.rb | 3 +-- spec/asana_find_release_task_action_spec.rb | 4 +-- spec/ddg_apple_automation_helper_spec.rb | 18 +++++++------ 13 files changed, 46 insertions(+), 33 deletions(-) diff --git a/lib/fastlane/plugin/ddg_apple_automation/actions/asana_find_release_task_action.rb b/lib/fastlane/plugin/ddg_apple_automation/actions/asana_find_release_task_action.rb index 6527491..c06c1d0 100644 --- a/lib/fastlane/plugin/ddg_apple_automation/actions/asana_find_release_task_action.rb +++ b/lib/fastlane/plugin/ddg_apple_automation/actions/asana_find_release_task_action.rb @@ -44,7 +44,7 @@ def self.run(params) UI.user_error!("No release task found for version #{latest_marketing_version}") unless release_task_id release_task_url = Helper::AsanaHelper.asana_task_url(release_task_id) - release_branch = "release/#{latest_marketing_version}" + release_branch = Helper::DdgAppleAutomationHelper.release_branch_name(platform, latest_marketing_version) UI.success("Found #{latest_marketing_version} release task: #{release_task_url}") Helper::GitHubActionsHelper.set_output("release_branch", release_branch) diff --git a/lib/fastlane/plugin/ddg_apple_automation/actions/tag_release_action.rb b/lib/fastlane/plugin/ddg_apple_automation/actions/tag_release_action.rb index f18796b..17ad141 100644 --- a/lib/fastlane/plugin/ddg_apple_automation/actions/tag_release_action.rb +++ b/lib/fastlane/plugin/ddg_apple_automation/actions/tag_release_action.rb @@ -28,11 +28,11 @@ def self.setup_constants(platform) end def self.run(params) - other_action.ensure_git_branch(branch: "^(:?release|hotfix)/.+$") + platform = params[:platform] || Actions.lane_context[Actions::SharedValues::PLATFORM_NAME] + other_action.ensure_git_branch(branch: "^(release|hotfix)/#{platform}/.+$") Helper::GitHelper.setup_git_user - params[:platform] ||= Actions.lane_context[Actions::SharedValues::PLATFORM_NAME] - setup_constants(params[:platform]) + setup_constants(platform) tag_and_release_output = create_tag_and_github_release(params[:is_prerelease], params[:github_token]) Helper::GitHubActionsHelper.set_output("tag", tag_and_release_output[:tag]) diff --git a/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete-with-tasks.html.erb b/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete-with-tasks.html.erb index d03e051..47f8090 100644 --- a/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete-with-tasks.html.erb +++ b/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete-with-tasks.html.erb @@ -1,8 +1,8 @@ - Build <%= tag %> is now available for internal testing through Sparkle and TestFlight.
+ Build <%= tag %> is now available for internal testing through <% if defined?(dmg_url) && !dmg_url.to_s.strip.empty? %>Sparkle and<% end %> TestFlight.

Added in this release: <%= tasks_since_last_internal_release %>

- 📥 DMG download link + <% if defined?(dmg_url) && !dmg_url.to_s.strip.empty? %>📥 DMG download link<% end %> \ No newline at end of file diff --git a/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete.html.erb b/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete.html.erb index 9055d2a..8640c21 100644 --- a/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete.html.erb +++ b/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-complete.html.erb @@ -1,5 +1,4 @@ - Build <%= tag %> is now available for internal testing through Sparkle and TestFlight.
-
- 📥 DMG download link + Build <%= tag %> is now available for internal testing through <% if defined?(dmg_url) && !dmg_url.to_s.strip.empty? %>Sparkle and<% end %> TestFlight.
+ <% if defined?(dmg_url) && !dmg_url.to_s.strip.empty? %>
📥 DMG download link<% end %> \ No newline at end of file diff --git a/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-ready-merge-failed.html.erb b/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-ready-merge-failed.html.erb index 52de680..1a82f27 100644 --- a/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-ready-merge-failed.html.erb +++ b/lib/fastlane/plugin/ddg_apple_automation/assets/asana_add_comment/templates/internal-release-ready-merge-failed.html.erb @@ -1,7 +1,7 @@

[ACTION NEEDED] Internal release build <%= tag %> ready