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

fixes linter peer dependencies #380

Merged
merged 1 commit into from
Feb 7, 2017
Merged

Conversation

delambo
Copy link
Member

@delambo delambo commented Feb 7, 2017

Dependencies in ESLint and StyleLint extensions are supposed to be peerDependencies. As a follow-up task, I'll update the 0.4.0 docs.

Copy link
Contributor

@tizmagik tizmagik left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Always a fan of semantic dependency defs! 😄 👍

@ccpricenytimes ccpricenytimes merged commit 4adf17b into master Feb 7, 2017
@ccpricenytimes ccpricenytimes deleted the fixes-linter-peers branch February 7, 2017 18:46
delambo added a commit that referenced this pull request Feb 7, 2017
* master:
  fixes linter peer dependencies (#380)
tizmagik added a commit that referenced this pull request Feb 17, 2017
* master: (26 commits)
  Expand version range of yarn for Travis (#400)
  update migration guide to 0.4.0 (#398)
  Update CHANGELOG.md
  adds 040 final version (#392)
  0.4.0 docs (#338)
  adds yarn version check to bootstrap (#391)
  Making kyt a dependency rather than a devDependency (#387)
  adds more functional tests (#388)
  rc5 (#384)
  fixes stylelintrc glob so that it respects file extensions (#383)
  prep work for rc3 (#381)
  Interactive setup (#378)
  Update webpack.base.js (#365)
  Adds monorepo bootstrap (#377)
  fixes linter peer dependencies (#380)
  Fix typo (#379)
  kyt-cli version bump (#376)
  updates the kyt-cli version (#370)
  fixes cli starter kyt path bug (#369)
  Adds prep for 0.4.0 release candidate (#368)
  ...
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.

3 participants