Skip to content
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

Stylelint config #216

Merged
merged 2 commits into from Nov 14, 2016
Merged

Stylelint config #216

merged 2 commits into from Nov 14, 2016

Conversation

planktonic
Copy link
Member

@planktonic planktonic commented Nov 9, 2016

Summary

WIP - wait for updates to the stylelint-config-bitcrowd, and update the version here

Fixes #63 .

The following changes are contained in this pull request:

  • Added the new bitcrowd stylelint config via npm, removed the old local version.
  • Fixed a few linting errors — linting that package had resulted in fixing some of the apparently-broken sass linting rules.

Changes to objects & trumps

  • Styleguide documentation has been updated.
  • backstop.json file has been updated
  • npm run test:build script has been sucessfully run and new/changed images committed
  • npm run stats script run and updated stats/css.json commited

Linting and testing

  • npm run test script has been run and visual tests pass
  • npm run checks script has been run without errors

@planktonic planktonic force-pushed the issue/63-stylelint-config branch 2 times, most recently from 7d7ad65 to e147679 Compare November 14, 2016 14:15
…l version.

Fixed a few linting errors — linting that package had resulted in fixing some of the apparently-broken sass linting rules.
@planktonic
Copy link
Member Author

Updated: now uses the most recent version of stylelint-config-bitcrowd.

@planktonic planktonic merged commit bfe76ad into master Nov 14, 2016
@planktonic planktonic deleted the issue/63-stylelint-config branch November 14, 2016 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

stylelint config
1 participant