Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed: frozen string literal #564

Closed
wants to merge 2 commits into from
Closed

Conversation

sa-progress
Copy link
Contributor

@sa-progress sa-progress commented Jan 3, 2024

Description

Please describe what this change achieves. Ensure you have read the Contributing to InSpec GCP document before
submitting.

This pull request addresses the removal of # frozen_string_literal: false from our Ruby codebase during the migration from Ruby 2.6 to Ruby 3.1. The Ruby 3.1 version brings improvements and changes to the language, and we are aligning our codebase with the latest practices.

Issues Resolved

List any existing issues this PR resolves, or any Discourse or StackOverflow discussion that's relevant.

  1. Migration to Ruby 3.1:
    Updated Ruby version in the project to 3.1.

  2. Removal of # frozen_string_literal: false:
    Removed instances of # frozen_string_literal: false from source files.
    Leveraged the default behaviour of frozen string literals in Ruby 3.1.

Please ensure commits have been signed-off for the Developer Certificate of Origin. See https://github.com/chef/chef/blob/master/CONTRIBUTING.md#developer-certification-of-origin-dco

Signed-off-by: Samir <85890442+sa-progress@users.noreply.github.com>
@sa-progress sa-progress requested a review from a team as a code owner January 3, 2024 16:02
@sa-progress sa-progress changed the title Removed: frozen string litral Removed: frozen string literal Jan 3, 2024
Signed-off-by: Samir <85890442+sa-progress@users.noreply.github.com>
@sa-progress sa-progress closed this Feb 6, 2024
@sa-progress sa-progress deleted the sa/cleanup_rubocop branch February 6, 2024 14:17
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