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.

Sign up for GitHub

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

Jump to bottom

Add stylelint to our automated testing #118

Closed
humphd opened this issue Nov 9, 2019 · 1 comment · Fixed by #125
Closed

Add stylelint to our automated testing #118

humphd opened this issue Nov 9, 2019 · 1 comment · Fixed by #125
Assignees
@cindyledev
Labels
area: CI/CD Continuous integration / Continuous delivery area: front-end
Projects
Main

Comments

@humphd
Copy link
Contributor

humphd commented Nov 9, 2019

Some people are beginning to experiment with CSS, and it would be good to add linting for it, to catch typos and other bugs (#117 is an example that we could catch via automation).

Let's add stylelint similar to eslint, so it can be run automatically as part of npm test.
@humphd humphd mentioned this issue Nov 9, 2019
Closed
@humphd humphd added area: CI/CD Continuous integration / Continuous delivery area: front-end labels Nov 9, 2019
@Reza-Rajabi Reza-Rajabi added this to Issue in Main via automation Nov 9, 2019
@Reza-Rajabi Reza-Rajabi moved this from Issue to To do in Main Nov 9, 2019
@cindyledev
Copy link
Contributor

Hi, I would like to work on this

@cindyledev cindyledev self-assigned this Nov 9, 2019
@cindyledev cindyledev mentioned this issue Nov 9, 2019
Merged
Main automation moved this from To do to Closed Nov 11, 2019
Reza-Rajabi added a commit that referenced this issue Nov 11, 2019
@Reza-Rajabi
Merge pull request #125 from cindyledev/issue-118
cca2a01 
Fixes #118: added stylelint
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@cindyledev cindyledev

Labels
area: CI/CD Continuous integration / Continuous delivery area: front-end
Projects
No open projects
Main
Closed
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Fixes #118: added stylelint cindyledev/telescope
3 participants
@humphd @Reza-Rajabi @cindyledev