Skip to content

fix: add frozen_string_literal and apt_update to test recipes#56

Open
damacus wants to merge 2 commits intomainfrom
fix/modernize-standards
Open

fix: add frozen_string_literal and apt_update to test recipes#56
damacus wants to merge 2 commits intomainfrom
fix/modernize-standards

Conversation

@damacus
Copy link
Member

@damacus damacus commented Mar 19, 2026

  • 11 files missing frozen_string_literal: true
  • 1 test recipe missing apt_update

…t_update

- Replace legacy Spork spec_helper with modern ChefSpec

- Add github_deploy resource spec (3 examples, 0 failures)

- Add frozen_string_literal to 11 files

- Add apt_update to test recipe

- Replace all EOL platforms with current non-EOL in kitchen.yml, kitchen.dokken.yml, CI
@damacus damacus force-pushed the fix/modernize-standards branch from e67fa33 to 96311f2 Compare March 19, 2026 11:22
@damacus damacus requested a review from a team as a code owner March 19, 2026 11:22
@damacus damacus force-pushed the fix/modernize-standards branch 2 times, most recently from 0ac93cd to 8b718da Compare March 19, 2026 11:35
- Pin release workflow to @main (was @5.0.8)

- Add slack_bot_token and slack_channel_id secrets

- Add .rubocop.yml with cookstyle require
@damacus damacus force-pushed the fix/modernize-standards branch from 8b718da to f27a844 Compare March 19, 2026 11:38
@github-actions
Copy link

Slowest examples

Top 3 slowest examples (0.14 seconds, 100% of total time)
Example Description Time in seconds
spec/resources/deploy_spec.rb:19 github_deploy deploy action is expected to create directory "/opt/deployments/test-repo/releases" 0.06869
spec/resources/deploy_spec.rb:21 github_deploy deploy action is expected to create link "/opt/deployments/test-repo/current" 0.03903
spec/resources/deploy_spec.rb:20 github_deploy deploy action is expected to create directory "/opt/deployments/test-repo/shared" 0.0387

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant