Sign upLogin

codeforsanjose/codeforsanjose

View on GitHub
Star
app/controllers/admin/application_controller.rb

Summary

Maintainability
A
0 mins
Test Coverage

TODO found
Open

    # TODO Add authentication logic here.
Severity: Minor
Found in app/controllers/admin/application_controller.rb by fixme

Annotation keywords like TODO should be all upper case, followed by a colon, and a space, then a note describing the problem.
Open

    # TODO Add authentication logic here.
Severity: Minor
Found in app/controllers/admin/application_controller.rb by rubocop

This cop checks that comment annotation keywords are written according to guidelines.

Example:

# bad
# TODO make better

# good
# TODO: make better

# bad
# TODO:make better

# good
# TODO: make better

# bad
# fixme: does not work

# good
# FIXME: does not work

# bad
# Optimize does not work

# good
# OPTIMIZE: does not work

Use nested module/class definitions instead of compact style.
Open

class Admin::ApplicationController < Administrate::ApplicationController
Severity: Minor
Found in app/controllers/admin/application_controller.rb by rubocop

This cop checks the style of children definitions at classes and modules. Basically there are two different styles:

Example: EnforcedStyle: nested (default)

# good
# have each child on its own line
class Foo
  class Bar
  end
end

Example: EnforcedStyle: compact

# good
# combine definitions as much as possible
class Foo::Bar
end

The compact style is only forced for classes/modules with one child.

There are no issues that match your filters.

Category
Status