Skip to content
This repository has been archived by the owner. It is now read-only.

Add linting #27

Merged
merged 4 commits into from Feb 17, 2016
Merged

Add linting #27

merged 4 commits into from Feb 17, 2016

Conversation

@fofr
Copy link
Contributor

@fofr fofr commented Feb 15, 2016

  • Add govuk-lint and jenkins task
  • Fix sass linting offences
  • Auto-correct ruby linting offences
  • Fix one offence that couldn't be auto-corrected: Put optional argument at end of expect_component
benlovell added a commit that referenced this pull request Feb 17, 2016
Add linting
@benlovell benlovell merged commit ce8139f into master Feb 17, 2016
1 check passed
1 check passed
default Build #95 succeeded on Jenkins
Details
@benlovell benlovell deleted the linting branch Feb 17, 2016
@jamiecobbett
Copy link

@jamiecobbett jamiecobbett commented Feb 17, 2016

👍

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.