-
Notifications
You must be signed in to change notification settings - Fork 309
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.
Already on GitHub? Sign in to your account
Dev --> Master #2142
Dev --> Master #2142
Conversation
- Install accesslint-ci and accesslint-cli. - Update _config-blog.yml by running `ruby config_blog.rb`. - Define a `serve-accesslint` script. - Use config-blog.yml to only serve the 3 most recent posts. - Compare errors with the most recent successful build of `dev` branch.
Serve accesslint
Handle author with no blog posts
Blog search filtering
Removed anti-aliasing from body
module SiteData | ||
class AuthorData | ||
def initialize | ||
@path = File.join(Dir.pwd, "_authors") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Prefer single-quoted strings when you don't need string interpolation or special symbols.
if x.data['name'].nil? | ||
puts "No such author: #{input} in #{x}" | ||
else | ||
x.data['name'].downcase == input.downcase |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use casecmp instead of downcase ==.
I found 4 accessibility errors:
|
@path = File.join(Dir.pwd, '_authors') | ||
end | ||
|
||
def update(author_file, key, value) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Assignment Branch Condition size for update is too high. [16.28/15]
Method has too many lines. [14/10]
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The alternative was also problematic !
I found 3 accessibility errors:
|
One small change and then this should be good to ship! Making a PR now |
Actually, nevermind @coreycaitlin. I was looking at the wrong branch. This looks good to go! |
馃槑 PREVIEW
Changes proposed in this pull request:
/cc @gemfarmer @elainekamlley