Merge pull request #12 from hmrc/revert-11-TRG-1455-2

Revert "TRG-1455: Pull in latest GDS changes (v3.5.0 + main) to HMRC fork."

Author: eddgrant

.github/workflows/publish.yaml

@@ -16,9 +16,7 @@ jobs:
       go: ${{ steps.gem_version.outputs.new_version }}
 
     steps:
-      - uses: actions/checkout@v4
-        with:
-          show-progress: false
+      - uses: actions/checkout@v3
 
       - uses: ruby/setup-ruby@v1
         with:
@@ -45,14 +43,12 @@ jobs:
       contents: write  # needed to be able to tag the release
     runs-on: ubuntu-latest
     needs: pre
-    if: needs.pre.outputs.go == 'true'
+    if: ${{ needs.pre.outputs.go == 'true' }}
 
     steps:
-      - uses: actions/checkout@v4
-        with:
-          show-progress: false
+      - uses: actions/checkout@v3
 
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v3
         with:
           node-version-file: '.nvmrc'
           cache: 'npm'

.github/workflows/test.yaml

@@ -1,11 +1,6 @@
 name: Test
 
-on: 
-  push:
-    branches:
-      - main
-  pull_request:
-  workflow_dispatch:
+on: [push, pull_request]
 
 jobs:
   test:
@@ -14,14 +9,12 @@ jobs:
 
     strategy:
       matrix:
-        ruby: ['3.1', '3.2', '3.3']
+        ruby: ['2.7', '3.2']
 
     steps:
-      - uses: actions/checkout@v4
-        with:
-          show-progress: false
+      - uses: actions/checkout@v3
 
-      - uses: actions/setup-node@v4
+      - uses: actions/setup-node@v3
         with:
           node-version-file: '.nvmrc'
           cache: 'npm'

.rubocop.yml

@@ -3,7 +3,7 @@ inherit_gem:
     - config/default.yml
 
 AllCops:
-  TargetRubyVersion: 3.1
+  TargetRubyVersion: 2.7
 
 Layout/HeredocIndentation:
   Enabled: false

CHANGELOG.md

@@ -2,10 +2,6 @@
 
 ## Unreleased
 
-- BREAKING: drop support for end-of-life Ruby versions 2.7 and 3.0. The minimum Ruby version is now 3.1.
-- Update gem dependencies.
-- Declare some missing indirect dependencies to prepare for Ruby 3.4. This also resolves some warnings about this at build time.
-
 ## 3.5.0
 
 ### New features

govuk_tech_docs.gemspec

@@ -33,30 +33,26 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^exe/}) { |f| File.basename(f) }
   spec.require_paths = %w[lib]
 
-  spec.required_ruby_version = ">= 3.1.0"
+  spec.required_ruby_version = ">= 2.7.0"
 
   spec.add_dependency "autoprefixer-rails", "~> 10.2"
-  spec.add_dependency "base64" # TODO: remove once middleman-sprockets declares this itself.
-  spec.add_dependency "bigdecimal" # TODO: remove once activesupport declares this itself.
   spec.add_dependency "chronic", "~> 0.10.2"
-  spec.add_dependency "csv" # TODO: remove once tilt declares this itself.
-  spec.add_dependency "haml", "~> 6.0"
+  spec.add_dependency "haml", "< 6.0.0"
   spec.add_dependency "middleman", "~> 4.0"
-  spec.add_dependency "middleman-autoprefixer", "~> 2.10"
-  spec.add_dependency "middleman-compass", "~> 4.0"
+  spec.add_dependency "middleman-autoprefixer", "~> 2.10.0"
+  spec.add_dependency "middleman-compass", ">= 4.0.0"
   spec.add_dependency "middleman-livereload"
   spec.add_dependency "middleman-search-gds"
   spec.add_dependency "middleman-sprockets", "~> 4.0.0"
-  spec.add_dependency "middleman-syntax", "~> 3.4"
-  spec.add_dependency "mutex_m" # TODO: remove once activesupport declares this itself.
+  spec.add_dependency "middleman-syntax", "~> 3.2.0"
   spec.add_dependency "nokogiri"
   spec.add_dependency "openapi3_parser", "~> 0.9.0"
-  spec.add_dependency "redcarpet", "~> 3.6"
+  spec.add_dependency "redcarpet", "~> 3.5.1"
 
   spec.add_development_dependency "byebug"
   spec.add_development_dependency "capybara", "~> 3.32"
-  spec.add_development_dependency "jasmine", "~> 3.5"
+  spec.add_development_dependency "jasmine", "~> 3.5.0"
   spec.add_development_dependency "rake", "~> 13.0"
-  spec.add_development_dependency "rspec", "~> 3.9"
-  spec.add_development_dependency "rubocop-govuk", "~> 4.10"
+  spec.add_development_dependency "rspec", "~> 3.9.0"
+  spec.add_development_dependency "rubocop-govuk", "~> 4.10.0"
 end

lib/govuk_tech_docs.rb

@@ -44,7 +44,7 @@ def self.configure(context, options = {})
                 renderer: TechDocsHTMLRenderer.new(
                   with_toc_data: true,
                   api: true,
-                  context:,
+                  context: context,
                 ),
                 fenced_code_blocks: true,
                 tables: true,

lib/govuk_tech_docs/redirects.rb

@@ -5,7 +5,7 @@ module GovukTechDocs
   class Redirects
     include GovukTechDocs::PathHelpers
 
-    LEADING_SLASH = %r{\A/}
+    LEADING_SLASH = %r{\A/}.freeze
 
     def initialize(context)
       @context = context

lib/govuk_tech_docs/table_of_contents/heading_tree_builder.rb

@@ -11,7 +11,7 @@ def tree
         @headings.each do |heading|
           move_to_depth(heading.size)
 
-          @pointer.children << HeadingTree.new(parent: @pointer, heading:)
+          @pointer.children << HeadingTree.new(parent: @pointer, heading: heading)
         end
 
         @tree

lib/govuk_tech_docs/table_of_contents/helpers.rb

@@ -12,7 +12,7 @@ module Helpers
 
       def single_page_table_of_contents(html, url: "", max_level: nil)
         output = "<ul>\n"
-        output += list_items_from_headings(html, url:, max_level:)
+        output += list_items_from_headings(html, url: url, max_level: max_level)
         output += "</ul>\n"
 
         output
@@ -36,7 +36,7 @@ def list_items_from_headings(html, url: "", max_level: nil)
         end
 
         tree = HeadingTreeBuilder.new(headings).tree
-        HeadingTreeRenderer.new(tree, max_level:).html
+        HeadingTreeRenderer.new(tree, max_level: max_level).html
       end
 
       def render_page_tree(resources, current_page, config, current_page_html)