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

feat(): install covalent tools as dependency #1199

Merged
merged 2 commits into from
Jul 19, 2018
Merged

Conversation

emoralesb05
Copy link
Contributor

@emoralesb05 emoralesb05 commented Jul 18, 2018

Description

this is to leverage the global tslint and stylelint files across
all our product portfolio

What's included?

  • added @covalent/tools as dependency
  • point to covalent tools tslint file
  • point to covalent tools stylelint file

Test Steps

  • yarn install
  • npm run tslint should work the same way (try forcing linting errors)
  • npm run stylelint should work the same way (try forcing linting errors)
  • ./node_modules/.bin/cov versions should work

General Tests for Every PR

  • npm run serve:prod still works.
  • npm run tslint passes.
  • npm run stylelint passes.
  • npm test passes and code coverage is not lower.
  • npm run build:lib still works.

this is to leverage the global tslint and stylelint files across
all our product portafolio
@emoralesb05 emoralesb05 self-assigned this Jul 18, 2018
@jeremysmartt jeremysmartt added this to the Release 2.0.0 milestone Jul 18, 2018
"max-empty-lines": 1,
"media-feature-name-no-unknown": true
}
"extends": "./node_modules/@covalent/tools/lint/stylelint/.stylelintrc.json"
Copy link
Collaborator

Choose a reason for hiding this comment

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

awesome

@emoralesb05 emoralesb05 merged commit 2c74ee6 into develop Jul 19, 2018
@emoralesb05 emoralesb05 deleted the feature/cov-tools branch July 19, 2018 00:12
kriswinbush pushed a commit to kriswinbush/covalent that referenced this pull request Feb 20, 2020
* feat(): install covalent tools as dependency

this is to leverage the global tslint and stylelint files across
all our product portafolio

* chore(): update to tools to beta.2
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.

2 participants