myles/jekyll-typogrify

View on GitHub

Showing 94 of 94 total issues

Method has too many lines. [15/10]
Open

def custom_caps(text)
# $1 and $2 are excluded HTML tags, $3 is the part before the caps and $4 is the caps match
text.gsub(%r{
(<[^/][^>]*?>)| # Ignore any opening tag, so we don't mess up attribute values
(\s|&nbsp;|^|'|"|>|) # Make sure our capture is preceded by whitespace or quotes
Severity: Minor
Found in lib/jekyll/typogrify.rb by rubocop

Jekyll::TypogrifyFilter has no descriptive comment
Open

module TypogrifyFilter
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#custom_caps doesn't depend on instance state (maybe move it to another class?)
Open

def custom_caps(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#caps doesn't depend on instance state (maybe move it to another class?)
Open

def caps(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#smartypants doesn't depend on instance state (maybe move it to another class?)
Open

def smartypants(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#widont doesn't depend on instance state (maybe move it to another class?)
Open

def widont(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#entities doesn't depend on instance state (maybe move it to another class?)
Open

def entities(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#emdash doesn't depend on instance state (maybe move it to another class?)
Open

def emdash(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#letter_spacing doesn't depend on instance state (maybe move it to another class?)
Open

def letter_spacing(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#amp doesn't depend on instance state (maybe move it to another class?)
Open

def amp(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#improve doesn't depend on instance state (maybe move it to another class?)
Open

def improve(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#initial_quotes doesn't depend on instance state (maybe move it to another class?)
Open

def initial_quotes(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Jekyll::TypogrifyFilter#titlecase doesn't depend on instance state (maybe move it to another class?)
Open

def titlecase(text)
Severity: Minor
Found in lib/jekyll/typogrify.rb by reek

Prefer single-quoted strings when you don't need string interpolation or special symbols.
Open

spec.homepage = "http://myles.github.io/jekyll-typogrify/"
Severity: Minor
Found in jekyll-typogrify.gemspec by rubocop

Prefer single-quoted strings when you don't need string interpolation or special symbols.
Open

VERSION = "0.3.5.pre"
Severity: Minor
Found in lib/jekyll/typogrify/version.rb by rubocop

Line is too long. [93/80]
Open

# Do nothing with the contents if ignored tags, the inside of an opening HTML element
Severity: Minor
Found in lib/jekyll/typogrify.rb by rubocop

Redundant return detected.
Open

return Typogruby.initial_quotes(text.to_s)
Severity: Minor
Found in lib/jekyll/typogrify.rb by rubocop

Dependencies should be sorted in an alphabetical order within their section of the gemspec. Dependency jekyll should appear before rake.
Open

spec.add_development_dependency "jekyll"
Severity: Minor
Found in jekyll-typogrify.gemspec by rubocop

Line is too long. [97/80]
Open

spec.description = %q{A Jekyll plugin that improves the typography of your Liquid templates.}
Severity: Minor
Found in jekyll-typogrify.gemspec by rubocop

Final newline missing.
Open

task :default => :spec
Severity: Minor
Found in Rakefile by rubocop
Severity
Category
Status
Source
Language