Skip to content
This repository was archived by the owner on Sep 25, 2018. It is now read-only.

Add CodeClimate configuration and Stylelint#45

Open
puregarlic wants to merge 10 commits intodevfrom
tool/codeclimate
Open

Add CodeClimate configuration and Stylelint#45
puregarlic wants to merge 10 commits intodevfrom
tool/codeclimate

Conversation

@puregarlic
Copy link
Member

It was much easier to just add both tools at the same time, so here they are. I have:

  • Added Stylelint dependencies and a script
  • Added a CodeClimate configuration file
  • Changed the testing server to build the application at runtime
  • Changed the test script from test:integration to just test
  • Swapped the Codacy badge with a CodeClimate badge

Adds stylelint, stylelint-config-standard, and stylelint-processor-html
Adds a configuration for Stylelint so it can actually be used
Changes the test:lint script into lint:js for finer control over scripts. Also changes
test:integration into just test, and modifies the CircleCI config to address this change.
It's much easier for the testing server to just build the application before launching tests, so
I've added this to the launch server.
Adds an initial configuration for CodeClimate, which should run ESLint, FIXME, and Stylelint on our
code at PR time.
Replaces the Codacy badge with a CodeClimate badge
CodeClimate has a whitelisted set of plugins, so I've added eslint-plugin-html as a fallback for
eslint-plugin-vue. This includes adding the dependency, registering the .vue extension in the
CodeClimate config, and adding the plugin to the ESLint configuration.
Complies with CodeClimate's eval of PR 45
@puregarlic puregarlic changed the title WIP: Add CodeClimate configuration and Stylelint Add CodeClimate configuration and Stylelint Dec 24, 2017
Copy link
Collaborator

@astra137 astra137 left a comment

Choose a reason for hiding this comment

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

👌

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants