Project

General

Profile

Bug #16564

"Missing frozen string literal comment." rubocop violations on develop

Added by Dominic Cleal over 3 years ago. Updated over 1 year ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
Tests
Target version:
Difficulty:
Triaged:
Bugzilla link:
Fixed in Releases:
Found in Releases:

Description

All files are failing rubocop checks since #16491 with the following error:

app/models/host.rb:1:1: C: Missing frozen string literal comment.
module Host
^

The Style/FrozenStringLiteralComment cop should be disabled or fixed.

There are also failures in config/initializers/secure_headers.rb that aren't passing, probably from when the PR was merged onto develop without re-checking it passed on the previous commit.


Related issues

Related to Foreman - Bug #16491: upgrade rubocop to version 0.42Closed2016-09-08
Related to Katello - Bug #16676: "Missing frozen string literal comment." rubocop violationsClosed2016-09-23

Associated revisions

Revision e1c55843 (diff)
Added by Tomer Brisker over 3 years ago

Fixes #16564 - Disable FrozenStringLiteralComment cop

Also fixes a failng cop in config/initializers/secure_headers.rb

History

#1 Updated by Dominic Cleal over 3 years ago

  • Related to Bug #16491: upgrade rubocop to version 0.42 added

#2 Updated by Tomer Brisker over 3 years ago

  • Status changed from New to Assigned
  • Assignee set to Tomer Brisker

#3 Updated by The Foreman Bot over 3 years ago

  • Status changed from Assigned to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/3846 added

#4 Updated by Anonymous over 3 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by Dominic Cleal over 3 years ago

  • Legacy Backlogs Release (now unused) set to 189

#6 Updated by Daniel Lobato Garcia over 3 years ago

  • Target version set to 1.5.2

#7 Updated by David Davis over 3 years ago

  • Related to Bug #16676: "Missing frozen string literal comment." rubocop violations added

Also available in: Atom PDF