-
Notifications
You must be signed in to change notification settings - Fork 311
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
dev --> master #1990
dev --> master #1990
Conversation
Update ruby version to 2.3.1
Add pa11y-ci
* Documentation on Contributing and Readme
Docker support(Update)
Transition from team to archive pages
New homepage Content Note: `Pa11yci` is failing to run in CircleCI but didn't find any accessibility issues when run locally.
Added Mark Headd to list of authors, team members
[WIP] Add markdown extension, update jekyll plugin documentation
|
||
module Jekyll | ||
module Converters | ||
class Markdown::RedcarpetExt |
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 nested module/class definitions instead of compact style.
end | ||
|
||
def self.add(item) | ||
if @current_headers |
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 2 (not 3) spaces for indentation.
@current_headers | ||
end | ||
|
||
def self.add(item) |
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.
Inconsistent indentation detected.
end | ||
end | ||
|
||
def header(title, level) |
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.
Method has too many lines. [12/10]
super options.merge(config_options) | ||
end | ||
|
||
def strip_dashes(str) |
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.
Method has too many lines. [11/10]
It looks like all checks have passed |
Fixes issue(s) #1637, #1931, #1945, #1898, #1985
😎 PREVIEW
Changes proposed in this pull request:
/cc @gboone @mugizico @elainekamlley @coreycaitlin @awfrancisco