From 6e65acca5cf86bb716eea598a3011a586eec1f94 Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Thu, 2 Oct 2025 10:21:19 -0600 Subject: [PATCH 01/16] Set ruby '3.1.4' in Gemfile && `bundle install` --- Gemfile | 2 +- Gemfile.lock | 5 +++-- 2 files changed, 4 insertions(+), 3 deletions(-) diff --git a/Gemfile b/Gemfile index e66e5e7288..2a9aa8721d 100644 --- a/Gemfile +++ b/Gemfile @@ -2,7 +2,7 @@ source 'https://rubygems.org' -ruby '>= 3.0' +ruby '3.1.4' # ===========# # CORE RAILS # diff --git a/Gemfile.lock b/Gemfile.lock index 9c399339bb..b123dadad1 100644 --- a/Gemfile.lock +++ b/Gemfile.lock @@ -210,7 +210,8 @@ GEM faraday (>= 0.8) faraday-net_http (3.4.0) net-http (>= 0.5.0) - ffi (1.17.1) + ffi (1.17.1-arm64-darwin) + ffi (1.17.1-x86_64-linux-gnu) flag_shih_tzu (0.3.23) fog-aws (3.30.0) base64 (~> 0.2.0) @@ -696,7 +697,7 @@ DEPENDENCIES yard-tomdoc RUBY VERSION - ruby 3.0.4p208 + ruby 3.1.4p223 BUNDLED WITH 2.4.17 From 76a5301a04268ace794b495c7f2928d6c3a8a0f4 Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Thu, 2 Oct 2025 10:23:27 -0600 Subject: [PATCH 02/16] Remove `.ruby-version` from `.gitignore` --- .gitignore | 3 --- .ruby-version | 1 + 2 files changed, 1 insertion(+), 3 deletions(-) create mode 100644 .ruby-version diff --git a/.gitignore b/.gitignore index 5943ad35e2..b32c4f2b28 100644 --- a/.gitignore +++ b/.gitignore @@ -1,6 +1,3 @@ -# Ignore rbenv files -.ruby-version - # Ignore bundler config /.bundle diff --git a/.ruby-version b/.ruby-version new file mode 100644 index 0000000000..0aec50e6ed --- /dev/null +++ b/.ruby-version @@ -0,0 +1 @@ +3.1.4 From 8b95198184178a48b9e9796b79169a2c09b23b9b Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Thu, 2 Oct 2025 10:28:45 -0600 Subject: [PATCH 03/16] Use `.ruby-version` for Ruby version in CI Removed the hardcoded `ruby-version` from GitHub Actions workflows. - `ruby/setup-ruby` now reads the version from `.ruby-version`. - Avoids duplication and keeps CI consistent with local development --- .github/workflows/danger.yml | 1 - .github/workflows/mysql.yml | 1 - .github/workflows/postgres.yml | 1 - .github/workflows/rubocop.yml | 1 - 4 files changed, 4 deletions(-) diff --git a/.github/workflows/danger.yml b/.github/workflows/danger.yml index 8fbb6af429..41a7bf0bd0 100644 --- a/.github/workflows/danger.yml +++ b/.github/workflows/danger.yml @@ -18,7 +18,6 @@ jobs: - name: 'Install Ruby' uses: ruby/setup-ruby@v1 with: - ruby-version: ${{ env.RUBY_VERSION }} - name: 'Bundle Install' run: | diff --git a/.github/workflows/mysql.yml b/.github/workflows/mysql.yml index 08a335329b..7c25271528 100644 --- a/.github/workflows/mysql.yml +++ b/.github/workflows/mysql.yml @@ -19,7 +19,6 @@ jobs: # Install Ruby and run bundler - uses: ruby/setup-ruby@v1 with: - ruby-version: '3.0' bundler-cache: true # Install Node diff --git a/.github/workflows/postgres.yml b/.github/workflows/postgres.yml index 517fd35954..c1e44af294 100644 --- a/.github/workflows/postgres.yml +++ b/.github/workflows/postgres.yml @@ -36,7 +36,6 @@ jobs: # Install Ruby and run bundler - uses: ruby/setup-ruby@v1 with: - ruby-version: '3.0' bundler-cache: true ## - run: echo 'NODE_OPTIONS="--openssl-legacy-provider"' >> $GITHUB_ENV diff --git a/.github/workflows/rubocop.yml b/.github/workflows/rubocop.yml index dc92a01f4b..15c3fd9028 100644 --- a/.github/workflows/rubocop.yml +++ b/.github/workflows/rubocop.yml @@ -13,7 +13,6 @@ jobs: # Install Ruby and run bundler - uses: ruby/setup-ruby@v1 with: - ruby-version: '3.0' bundler-cache: true # Run the Rubocop linter checks From dc25eec413af3f7498a6e480445a243d3ba8c14f Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Thu, 2 Oct 2025 10:21:27 -0600 Subject: [PATCH 04/16] `bundle exec rubocop -A` --- app/controllers/plans_controller.rb | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/app/controllers/plans_controller.rb b/app/controllers/plans_controller.rb index 8f7c44393a..c66bbfdeb4 100644 --- a/app/controllers/plans_controller.rb +++ b/app/controllers/plans_controller.rb @@ -193,7 +193,8 @@ def show @default_orgs = Org.default_orgs @all_ggs_grouped_by_org.each do |org, ggs| # @default_orgs and already selected guidance groups are important. - if (@default_orgs.include?(org) || (ggs & @selected_guidance_groups).any?) && !@important_ggs.include?([org, ggs]) + if (@default_orgs.include?(org) || ggs.intersect?(@selected_guidance_groups)) && !@important_ggs.include?([org, + ggs]) @important_ggs << [org, ggs] end end From 3664410b63def606e549b49f1a83e513171b3220 Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Mon, 6 Oct 2025 10:24:45 -0600 Subject: [PATCH 05/16] Update CHANGELOG.md --- CHANGELOG.md | 3 +++ 1 file changed, 3 insertions(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 0633ac0360..74281a792e 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -1,5 +1,8 @@ # Changelog +## v5.0.2 +- Bump Ruby to v3.1.4 and use `.ruby-version` in CI [#3566](https://github.com/DMPRoadmap/roadmap/pull/3566) + ## v5.0.1 - Updated seeds.rb file for identifier_schemes to include context value and removed logo_url and idenitifier_prefix for Shibboleth (as it was causing issues with SSO). [#3525](https://github.com/DMPRoadmap/roadmap/pull/3525) - Adjustments to style of select tags and plan download layout [#3509](https://github.com/DMPRoadmap/roadmap/pull/3509) From 7c142ebd20c3f7d6e14ad2827f74887d93dce606 Mon Sep 17 00:00:00 2001 From: momo3404 Date: Mon, 6 Oct 2025 11:11:57 -0600 Subject: [PATCH 06/16] Enable session timeout after 90 mins of inactivity --- app/models/user.rb | 4 ++-- config/initializers/devise.rb | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/app/models/user.rb b/app/models/user.rb index e3ecccf039..94d0035eec 100644 --- a/app/models/user.rb +++ b/app/models/user.rb @@ -63,9 +63,9 @@ class User < ApplicationRecord # Devise # Include default devise modules. Others available are: # :token_authenticatable, :confirmable, - # :lockable, :timeoutable and :omniauthable + # :lockable and :omniauthable devise :invitable, :database_authenticatable, :registerable, :recoverable, - :rememberable, :trackable, :validatable, :omniauthable, + :rememberable, :trackable, :validatable, :omniauthable, :timeoutable, omniauth_providers: %i[shibboleth orcid] # default user language to the default language diff --git a/config/initializers/devise.rb b/config/initializers/devise.rb index 1cb771e8f6..8649665ccf 100755 --- a/config/initializers/devise.rb +++ b/config/initializers/devise.rb @@ -172,7 +172,7 @@ # ==> Configuration for :timeoutable # The time you want to timeout the user session without activity. After this # time the user will be asked for credentials again. Default is 30 minutes. - config.timeout_in = 3.hours + config.timeout_in = 90.minutes # If true, expires auth token on session timeout. # config.expire_auth_token_on_timeout = false From fc31d93b1b3ae2227926984d638ed463a86ee748 Mon Sep 17 00:00:00 2001 From: momo3404 Date: Mon, 6 Oct 2025 11:49:16 -0600 Subject: [PATCH 07/16] Add changelog entry --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 74281a792e..5c379dbc2f 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -2,6 +2,7 @@ ## v5.0.2 - Bump Ruby to v3.1.4 and use `.ruby-version` in CI [#3566](https://github.com/DMPRoadmap/roadmap/pull/3566) +- Enable session timeout after 90 minutes of inactivity [#3568](https://github.com/DMPRoadmap/roadmap/pull/3568) ## v5.0.1 - Updated seeds.rb file for identifier_schemes to include context value and removed logo_url and idenitifier_prefix for Shibboleth (as it was causing issues with SSO). [#3525](https://github.com/DMPRoadmap/roadmap/pull/3525) From 21c466e9f863456f6a012eb23f3697721933c029 Mon Sep 17 00:00:00 2001 From: momo3404 Date: Mon, 6 Oct 2025 12:05:24 -0600 Subject: [PATCH 08/16] Validate CSV Separator --- app/controllers/usage_controller.rb | 9 +++++++-- 1 file changed, 7 insertions(+), 2 deletions(-) diff --git a/app/controllers/usage_controller.rb b/app/controllers/usage_controller.rb index bf373aee17..f489a30130 100644 --- a/app/controllers/usage_controller.rb +++ b/app/controllers/usage_controller.rb @@ -151,9 +151,14 @@ def parse_filtered params[:filtered].present? && params[:filtered] == 'true' end - # set the csv separator or default to comma + # This sets the csv separator + # Ensures separator is either a comma or a safe separator def sep_param - params['sep'] || ',' + safe_csv_separators = Rails.configuration.x.application.csv_separators + sep = params['sep'].to_s + return sep if safe_csv_separators.include?(sep) + + ',' end def min_max_dates(args:) From c545f6dcc672fd735911a882b17c1d73989d8291 Mon Sep 17 00:00:00 2001 From: momo3404 Date: Mon, 6 Oct 2025 12:08:17 -0600 Subject: [PATCH 09/16] Add changelog entry --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 5c379dbc2f..af0f708510 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -3,6 +3,7 @@ ## v5.0.2 - Bump Ruby to v3.1.4 and use `.ruby-version` in CI [#3566](https://github.com/DMPRoadmap/roadmap/pull/3566) - Enable session timeout after 90 minutes of inactivity [#3568](https://github.com/DMPRoadmap/roadmap/pull/3568) +- Validate CSV Separator [#3569](https://github.com/DMPRoadmap/roadmap/pull/3569) ## v5.0.1 - Updated seeds.rb file for identifier_schemes to include context value and removed logo_url and idenitifier_prefix for Shibboleth (as it was causing issues with SSO). [#3525](https://github.com/DMPRoadmap/roadmap/pull/3525) From 0d275a22a157f5451f0ec984b59997200839e70e Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Wed, 23 Apr 2025 15:26:55 -0600 Subject: [PATCH 10/16] Fix rendering of `confirm_merge` partial - This change simplifies the partial naming by taking advantage of Rails' naming conventions and resolves the following error: ``` ActionView::MissingTemplate - Missing partial super_admin/users/_confirm_merge.html.erb with {:locale=>[:"en-CA", :en, :"en-GB"], :formats=>[:js, :html, :text, :css, :ics, :csv, :vcf, :vtt, :png, :jpeg, :gif, :bmp, :tiff, :svg, :webp, :mpeg, :mp3, :ogg, :m4a, :webm, :mp4, :otf, :ttf, :woff, :woff2, :xml, :rss, :atom, :yaml, :multipart_form, :url_encoded_form, :json, :pdf, :zip, :gzip, :docx, :turbo_stream], :variants=>[], :handlers=>[:raw, :erb, :html, :builder, :ruby, :jbuilder]}. ... app/controllers/super_admin/users_controller.rb:94:in `search' ``` - The prior `'super_admin/users/confirm_merge.html.erb'` code works with release`v4.2`, which uses Rails 6. However, it generates the following deprecation warning: ``` DEPRECATION WARNING: Rendering actions with '.' in the name is deprecated: super_admin/users/_confirm_merge.html.erb (called from search at /home/aaron/Documents/GitHub/roadmap_upstream/app/controllers/super_admin/users_controller.rb:94) ``` --- app/controllers/super_admin/users_controller.rb | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/app/controllers/super_admin/users_controller.rb b/app/controllers/super_admin/users_controller.rb index a27f528adb..c4b96141b7 100644 --- a/app/controllers/super_admin/users_controller.rb +++ b/app/controllers/super_admin/users_controller.rb @@ -91,7 +91,7 @@ def search # WHAT TO RETURN!?!?! if @users.present? # found a user, or Users, submit for merge render json: { - form: render_to_string(partial: 'super_admin/users/confirm_merge.html.erb') + form: render_to_string(partial: 'confirm_merge') } else # NO USER, re-render w/error? flash.now[:alert] = 'Unable to find user' From dc579d949c81837caf7d8f42e645307fccb28b00 Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Mon, 6 Oct 2025 14:04:04 -0600 Subject: [PATCH 11/16] Update CHANGELOG.md --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index af0f708510..873ebd8808 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -4,6 +4,7 @@ - Bump Ruby to v3.1.4 and use `.ruby-version` in CI [#3566](https://github.com/DMPRoadmap/roadmap/pull/3566) - Enable session timeout after 90 minutes of inactivity [#3568](https://github.com/DMPRoadmap/roadmap/pull/3568) - Validate CSV Separator [#3569](https://github.com/DMPRoadmap/roadmap/pull/3569) +- Fix rendering of `confirm_merge` partial [#3567](https://github.com/DMPRoadmap/roadmap/pull/3567) ## v5.0.1 - Updated seeds.rb file for identifier_schemes to include context value and removed logo_url and idenitifier_prefix for Shibboleth (as it was causing issues with SSO). [#3525](https://github.com/DMPRoadmap/roadmap/pull/3525) From c742a4ce5e5c7174b7c54c8f565776bc86f65728 Mon Sep 17 00:00:00 2001 From: o200 Date: Wed, 20 Aug 2025 16:33:55 +0700 Subject: [PATCH 12/16] configure brakeman --- .github/workflows/brakeman.yml | 24 +++++++++++------------- README.md | 2 +- 2 files changed, 12 insertions(+), 14 deletions(-) diff --git a/.github/workflows/brakeman.yml b/.github/workflows/brakeman.yml index 0a64ca04c0..899d3f56bc 100644 --- a/.github/workflows/brakeman.yml +++ b/.github/workflows/brakeman.yml @@ -1,23 +1,21 @@ name: Brakeman on: - pull_request: - branches: - master + push: + branches: main jobs: brakeman: - runs-on: ubuntu-24.04 steps: - - uses: actions/checkout@v2 + - uses: actions/checkout@v2 - # Will run Brakeman checks on dependencies - # https://github.com/marketplace/actions/brakeman-action - - name: Brakeman - uses: artplan1/brakeman-action@v1.2.1 - with: - flags: "--color" - env: - GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} + # Will run Brakeman checks on dependencies + # https://github.com/marketplace/actions/brakeman-action + - name: Brakeman + uses: artplan1/brakeman-action@v1.2.1 + with: + flags: "--color" + env: + GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }} diff --git a/README.md b/README.md index e0c2c366b9..50f9ababf1 100644 --- a/README.md +++ b/README.md @@ -1,6 +1,6 @@ ## DMP Roadmap -[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Brakeman/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) +[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Brakeman/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/brakeman.yml) [![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Rubocop/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) [![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/ESLint/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) [![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Tests%20-%20PostgreSQL/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) From c95c672c77ed2ab6e40f23583e8512860d78302b Mon Sep 17 00:00:00 2001 From: o200 Date: Thu, 21 Aug 2025 08:29:13 +0700 Subject: [PATCH 13/16] set 'Tests - PostgreSQL' on push, format --- .github/workflows/postgres.yml | 170 ++++++++++++++++----------------- 1 file changed, 85 insertions(+), 85 deletions(-) diff --git a/.github/workflows/postgres.yml b/.github/workflows/postgres.yml index c1e44af294..dbc9d73ba1 100644 --- a/.github/workflows/postgres.yml +++ b/.github/workflows/postgres.yml @@ -1,6 +1,6 @@ name: Tests - PostgreSQL -on: [pull_request] +on: [push, pull_request] jobs: postgresql: @@ -16,7 +16,7 @@ jobs: # variable to bypass this changes: # https://github.com/docker-library/postgres/issues/681 POSTGRES_HOST_AUTH_METHOD: trust - ports: ['5432:5432'] + ports: ["5432:5432"] options: >- --health-cmd pg_isready --health-interval 10s @@ -30,86 +30,86 @@ jobs: DISABLE_SPRING: 1 steps: - # Checkout the repo - - uses: actions/checkout@v3 - - # Install Ruby and run bundler - - uses: ruby/setup-ruby@v1 - with: - bundler-cache: true - - ## - run: echo 'NODE_OPTIONS="--openssl-legacy-provider"' >> $GITHUB_ENV - ## /home/runner/runners/2.301.1/externals/node12/bin/node: --openssl-legacy-provider is not allowed in NODE_OPTIONS - - # Install Node - - uses: actions/setup-node@v3 - with: - node-version: 18 - cache: 'yarn' - - # Install the Postgres developer packages - - name: 'Install Postgresql Packages' - run: | - sudo apt-get update - sudo apt-get install libpq-dev - - # Install ImageMagick (for `identify` command) - - name: 'Install ImageMagick' - run: | - sudo apt-get install -y imagemagick - - # Copy all of the example configs over - - name: 'Setup Default Configuration' - run: | - cp config/database.yml.sample config/database.yml - cp config/initializers/contact_us.rb.example config/initializers/contact_us.rb - cp config/initializers/wicked_pdf.rb.example config/initializers/wicked_pdf.rb - cp .env.postgresql .env - - # Stub out the Rails credentials file so that we can start the Rails app - - name: 'Setup Credentials' - run: | - # generate a default credential file and key - EDITOR="sh -c 'echo \"$(cat config/credentials.yml.postgresql)\" > \$1' --" bundle exec rails credentials:edit - - # Set the path to the wkhtmltopdf executable - - name: 'Determine wkhtmltopdf location' - run: echo "WICKED_PDF_PATH=`bundle exec which wkhtmltopdf`" >> $GITHUB_ENV - - # Run yarn install for JS dependencies - - name: 'Yarn Install' - run: | - yarn install - - # TEMPORARY WORKAROUND FOR THE FOLLOWING ISSUE: https://github.com/DMPRoadmap/roadmap/issues/3485 - # Remove this once our tests are compatible with the new version of Chrome - # Source: https://github.com/teamcapybara/capybara/issues/2800#issuecomment-2731100953 - - name: Remove image-bundled Chrome - run: sudo apt-get purge google-chrome-stable - - name: Setup stable Chrome - uses: browser-actions/setup-chrome@v1 - with: - chrome-version: 128 - install-chromedriver: true - install-dependencies: true - - # Initialize the DB - - name: 'Setup Test DB' - run: | - bundle exec rails db:setup - bundle exec rails db:migrate - - # Prebuild the CSS, JS and image assets - - name: 'Compile Assets' - run: bundle exec rails assets:precompile - - # Run the unit and functional tests - - name: 'Run Rspec Unit and Functional Tests' - run: | - bundle exec rspec spec/models/ spec/policies/ spec/services/ spec/helpers/ - bundle exec rspec spec/controllers/ spec/presenters/ spec/requests/ spec/views - bundle exec rspec spec/mixins/ - - # Run the time consuming integration tests (using Chrome headless browser) - - name: 'Run Integration Tests' - run: bundle exec rspec spec/features/ + # Checkout the repo + - uses: actions/checkout@v3 + + # Install Ruby and run bundler + - uses: ruby/setup-ruby@v1 + with: + bundler-cache: true + + ## - run: echo 'NODE_OPTIONS="--openssl-legacy-provider"' >> $GITHUB_ENV + ## /home/runner/runners/2.301.1/externals/node12/bin/node: --openssl-legacy-provider is not allowed in NODE_OPTIONS + + # Install Node + - uses: actions/setup-node@v3 + with: + node-version: 18 + cache: "yarn" + + # Install the Postgres developer packages + - name: "Install Postgresql Packages" + run: | + sudo apt-get update + sudo apt-get install libpq-dev + + # Install ImageMagick (for `identify` command) + - name: "Install ImageMagick" + run: | + sudo apt-get install -y imagemagick + + # Copy all of the example configs over + - name: "Setup Default Configuration" + run: | + cp config/database.yml.sample config/database.yml + cp config/initializers/contact_us.rb.example config/initializers/contact_us.rb + cp config/initializers/wicked_pdf.rb.example config/initializers/wicked_pdf.rb + cp .env.postgresql .env + + # Stub out the Rails credentials file so that we can start the Rails app + - name: "Setup Credentials" + run: | + # generate a default credential file and key + EDITOR="sh -c 'echo \"$(cat config/credentials.yml.postgresql)\" > \$1' --" bundle exec rails credentials:edit + + # Set the path to the wkhtmltopdf executable + - name: "Determine wkhtmltopdf location" + run: echo "WICKED_PDF_PATH=`bundle exec which wkhtmltopdf`" >> $GITHUB_ENV + + # Run yarn install for JS dependencies + - name: "Yarn Install" + run: | + yarn install + + # TEMPORARY WORKAROUND FOR THE FOLLOWING ISSUE: https://github.com/DMPRoadmap/roadmap/issues/3485 + # Remove this once our tests are compatible with the new version of Chrome + # Source: https://github.com/teamcapybara/capybara/issues/2800#issuecomment-2731100953 + - name: Remove image-bundled Chrome + run: sudo apt-get purge google-chrome-stable + - name: Setup stable Chrome + uses: browser-actions/setup-chrome@v1 + with: + chrome-version: 128 + install-chromedriver: true + install-dependencies: true + + # Initialize the DB + - name: "Setup Test DB" + run: | + bundle exec rails db:setup + bundle exec rails db:migrate + + # Prebuild the CSS, JS and image assets + - name: "Compile Assets" + run: bundle exec rails assets:precompile + + # Run the unit and functional tests + - name: "Run Rspec Unit and Functional Tests" + run: | + bundle exec rspec spec/models/ spec/policies/ spec/services/ spec/helpers/ + bundle exec rspec spec/controllers/ spec/presenters/ spec/requests/ spec/views + bundle exec rspec spec/mixins/ + + # Run the time consuming integration tests (using Chrome headless browser) + - name: "Run Integration Tests" + run: bundle exec rspec spec/features/ From 4c24099f0b9256dde170d58e0e6ac9c91a714f03 Mon Sep 17 00:00:00 2001 From: o200 Date: Thu, 21 Aug 2025 08:45:12 +0700 Subject: [PATCH 14/16] configure badges links, hide disabled "Tests - MySQL" --- README.md | 12 +++++++----- 1 file changed, 7 insertions(+), 5 deletions(-) diff --git a/README.md b/README.md index 50f9ababf1..4fbbaecb90 100644 --- a/README.md +++ b/README.md @@ -1,10 +1,12 @@ ## DMP Roadmap -[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Brakeman/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/brakeman.yml) -[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Rubocop/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) -[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/ESLint/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) -[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Tests%20-%20PostgreSQL/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) -[![Actions Status](https://github.com/DMPRoadmap/roadmap/workflows/Tests%20-%20MySQL/badge.svg)](https://github.com/DMPRoadmap/roadmap/actions) +[![Actions Status](https://github.com/DMPRoadmap/roadmap/actions/workflows/brakeman.yml/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/brakeman.yml) +[![Actions Status](https://github.com/DMPRoadmap/roadmap/actions/workflows/rubocop.yml/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/rubocop.yml) +[![Actions Status](https://github.com/DMPRoadmap/roadmap/actions/workflows/eslint.yml/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/eslint.yml) +[![Actions Status](https://github.com/DMPRoadmap/roadmap/actions/workflows/postgres.yml/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/postgres.yml) +[![Actions Status](https://github.com/DMPRoadmap/roadmap/actions/workflows/danger.yml/badge.svg?branch=main)](https://github.com/DMPRoadmap/roadmap/actions/workflows/danger.yml) + + DMP Roadmap is a Data Management Planning tool. Management and development of DMP Roadmap is jointly provided by the Digital Curation Centre (DCC), http://www.dcc.ac.uk/, and the University of California Curation Center (UC3), http://www.cdlib.org/services/uc3/. From 5ff1f02edfab9fecead21f824cabde0b616c57c0 Mon Sep 17 00:00:00 2001 From: o200 Date: Thu, 21 Aug 2025 13:56:42 +0700 Subject: [PATCH 15/16] made Brakeman to work on next-release/* branches --- .github/workflows/brakeman.yml | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/.github/workflows/brakeman.yml b/.github/workflows/brakeman.yml index 899d3f56bc..52ecd71cf1 100644 --- a/.github/workflows/brakeman.yml +++ b/.github/workflows/brakeman.yml @@ -2,7 +2,9 @@ name: Brakeman on: push: - branches: main + branches: + - main + - next-release/* jobs: brakeman: From fb48734b2c1c433e8e261f36b0ea43c647060070 Mon Sep 17 00:00:00 2001 From: aaronskiba Date: Tue, 7 Oct 2025 09:19:03 -0600 Subject: [PATCH 16/16] Update CHANGELOG.md --- CHANGELOG.md | 1 + 1 file changed, 1 insertion(+) diff --git a/CHANGELOG.md b/CHANGELOG.md index 873ebd8808..673d1a0446 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -5,6 +5,7 @@ - Enable session timeout after 90 minutes of inactivity [#3568](https://github.com/DMPRoadmap/roadmap/pull/3568) - Validate CSV Separator [#3569](https://github.com/DMPRoadmap/roadmap/pull/3569) - Fix rendering of `confirm_merge` partial [#3567](https://github.com/DMPRoadmap/roadmap/pull/3567) +- Improve CI configuration [#3551](https://github.com/DMPRoadmap/roadmap/issues/3551) ## v5.0.1 - Updated seeds.rb file for identifier_schemes to include context value and removed logo_url and idenitifier_prefix for Shibboleth (as it was causing issues with SSO). [#3525](https://github.com/DMPRoadmap/roadmap/pull/3525)