Fix RuboCop offenses and enable PR checks

[Copilot]

CI RuboCop job failing with 2 autocorrectable offenses in config/application.rb and config/boot.rb.

Changes

• config/application.rb:13 - Removed trailing whitespace after config.load_defaults 7.0
• config/boot.rb:9 - Changed alias_method :original_freeze, :freeze to alias original_freeze freeze
• .github/workflows/rubocop.yml - Updated workflow to run on pull requests targeting main branch

# Before
class Array
  alias_method :original_freeze, :freeze
  
  def freeze
    # ...
  end
end

# After  
class Array
  alias original_freeze freeze
  
  def freeze
    # ...
  end
end

The RuboCop workflow now runs on both pushes to main and pull requests targeting main, enabling earlier detection of style issues.

These are style-only changes with no behavioral impact.

This pull request was created as a result of the following prompt from Copilot chat.

CI RuboCop job is failing (job id 55257039222). Logs show 2 offenses (both autocorrectable) and one offending line appears as:

alias_method :original_freeze, :freeze

Goal: create a pull request that fixes the RuboCop offenses and restores the RuboCop CI job to pass.

Required changes (actionable, testable):

1. Run RuboCop auto-correct to fix all autocorrectable offenses and commit the resulting code changes. Preferred command: bundle exec rubocop -A.

2. Specifically replace uses of alias_method where RuboCop prefers alias keyword. For example, change:

alias_method :original_freeze, :freeze

to:

alias original_freeze freeze

Make these replacements in any files where they occur.

3. Ensure no behavior changes are introduced beyond style fixes. Preserve indentation and class/module structure.

4. Add a brief commit message: "Fix RuboCop offenses (autocorrected)".

5. Run bundle exec rubocop --display-cop-names locally (or in CI) to confirm zero offenses and include the confirmation in the PR description.

Notes / context:

• The failing workflow: .github/workflows/rubocop.yml at ref 8d42761 (the CI run that failed). See job id 55257039222 for logs.
• The two offenses shown are autocorrectable; use rubocop's auto-correct to ensure all changes are applied consistently.

Deliverable:

• A PR branch off main with the autocorrected changes applied and committed.
• PR title matching the problem title and description explaining the changes and the verification steps (rubocop run output confirming no offenses).

If multiple files are changed, group them under a single commit. Do not modify the CI workflow itself (except to re-run tests if necessary).

Original prompt

CI RuboCop job is failing (job id 55257039222). Logs show 2 offenses (both autocorrectable) and one offending line appears as:

alias_method :original_freeze, :freeze

Goal: create a pull request that fixes the RuboCop offenses and restores the RuboCop CI job to pass.

Required changes (actionable, testable):

1. Run RuboCop auto-correct to fix all autocorrectable offenses and commit the resulting code changes. Preferred command: bundle exec rubocop -A.

2. Specifically replace uses of alias_method where RuboCop prefers alias keyword. For example, change:

alias_method :original_freeze, :freeze

to:

alias original_freeze freeze

Make these replacements in any files where they occur.

3. Ensure no behavior changes are introduced beyond style fixes. Preserve indentation and class/module structure.

4. Add a brief commit message: "Fix RuboCop offenses (autocorrected)".

5. Run bundle exec rubocop --display-cop-names locally (or in CI) to confirm zero offenses and include the confirmation in the PR description.

Notes / context:

• The failing workflow: .github/workflows/rubocop.yml at ref 8d42761 (the CI run that failed). See job id 55257039222 for logs.
• The two offenses shown are autocorrectable; use rubocop's auto-correct to ensure all changes are applied consistently.

Deliverable:

• A PR branch off main with the autocorrected changes applied and committed.
• PR title matching the problem title and description explaining the changes and the verification steps (rubocop run output confirming no offenses).

If multiple files are changed, group them under a single commit. Do not modify the CI workflow itself (except to re-run tests if necessary).

This pull request was created as a result of the following prompt from Copilot chat.

CI RuboCop job is failing (job id 55257039222). Logs show 2 offenses (both autocorrectable) and one offending line appears as:

alias_method :original_freeze, :freeze

Goal: create a pull request that fixes the RuboCop offenses and restores the RuboCop CI job to pass.

Required changes (actionable, testable):

1. Run RuboCop auto-correct to fix all autocorrectable offenses and commit the resulting code changes. Preferred command: bundle exec rubocop -A.

2. Specifically replace uses of alias_method where RuboCop prefers alias keyword. For example, change:

alias_method :original_freeze, :freeze

to:

alias original_freeze freeze

Make these replacements in any files where they occur.

3. Ensure no behavior changes are introduced beyond style fixes. Preserve indentation and class/module structure.

4. Add a brief commit message: "Fix RuboCop offenses (autocorrected)".

5. Run bundle exec rubocop --display-cop-names locally (or in CI) to confirm zero offenses and include the confirmation in the PR description.

Notes / context:

• The failing workflow: .github/workflows/rubocop.yml at ref 8d42761 (the CI run that failed). See job id 55257039222 for logs.
• The two offenses shown are autocorrectable; use rubocop's auto-correct to ensure all changes are applied consistently.

Deliverable:

• A PR branch off main with the autocorrected changes applied and committed.
• PR title matching the problem title and description explaining the changes and the verification steps (rubocop run output confirming no offenses).

If multiple files are changed, group them under a single commit. Do not modify the CI workflow itself (except to re-run tests if necessary).

---

💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

[tuanle03]

@copilot rework github action to run rubocop in any pr merged into main

[Copilot]

@copilot rework github action to run rubocop in any pr merged into main

Updated the RuboCop workflow in commit 5180a79. It now runs on both pushes to main and pull requests targeting main. The workflow will execute on this PR to verify the RuboCop fixes.