Flockingbird/roost

View on GitHub
config/environment.rb

Summary

Maintainability
A
0 mins
Test Coverage
A
98%

Roost::Config#database_url is a writable attribute
Open

    attr_accessor :database_url, :secret_base, :web_url
Severity: Minor
Found in config/environment.rb by reek

A class that publishes a setter for an instance variable invites client classes to become too intimate with its inner workings, and in particular with its representation of state.

The same holds to a lesser extent for getters, but Reek doesn't flag those.

Example

Given:

class Klass
  attr_accessor :dummy
end

Reek would emit the following warning:

reek test.rb

test.rb -- 1 warning:
  [2]:Klass declares the writable attribute dummy (Attribute)

Roost::Config#web_url is a writable attribute
Open

    attr_accessor :database_url, :secret_base, :web_url
Severity: Minor
Found in config/environment.rb by reek

A class that publishes a setter for an instance variable invites client classes to become too intimate with its inner workings, and in particular with its representation of state.

The same holds to a lesser extent for getters, but Reek doesn't flag those.

Example

Given:

class Klass
  attr_accessor :dummy
end

Reek would emit the following warning:

reek test.rb

test.rb -- 1 warning:
  [2]:Klass declares the writable attribute dummy (Attribute)

Roost::Config#secret_base is a writable attribute
Open

    attr_accessor :database_url, :secret_base, :web_url
Severity: Minor
Found in config/environment.rb by reek

A class that publishes a setter for an instance variable invites client classes to become too intimate with its inner workings, and in particular with its representation of state.

The same holds to a lesser extent for getters, but Reek doesn't flag those.

Example

Given:

class Klass
  attr_accessor :dummy
end

Reek would emit the following warning:

reek test.rb

test.rb -- 1 warning:
  [2]:Klass declares the writable attribute dummy (Attribute)

There are no issues that match your filters.

Category
Status