Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

A tiny code cleanup #507

Merged
merged 1 commit into from

2 participants

@jlecour

No need to have a variable assignment, especially if it has the same name as the method parameter.

@mojombo mojombo merged commit 41f2805 into jekyll:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 11, 2012
  1. @jlecour

    No need to have a variable assignment, especially if it has the same …

    jlecour authored
    …name as the method parameter.
This page is out of date. Refresh to see the latest.
Showing with 1 addition and 1 deletion.
  1. +1 −1  lib/jekyll/site.rb
View
2  lib/jekyll/site.rb
@@ -310,7 +310,7 @@ def site_payload
#
# Returns the Array of filtered entries.
def filter_entries(entries)
- entries = entries.reject do |e|
+ entries.reject do |e|
unless self.include.include?(e)
['.', '_', '#'].include?(e[0..0]) ||
e[-1..-1] == '~' ||
Something went wrong with that request. Please try again.