Skip to content
This repository was archived by the owner on Apr 12, 2024. It is now read-only.

jscs style check #6062

Closed
wants to merge 5 commits into from
Closed

Conversation

IgorMinar
Copy link
Contributor

No description provided.

@@ -41,7 +41,9 @@
"grunt-shell": "~0.4.0",
"semver": "~2.1.0",
"lodash": "~2.1.0",
"browserstacktunnel-wrapper": "~1.1.1"
"browserstacktunnel-wrapper": "~1.1.1",
"jscs": "~1.2.4",
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to add this explicitly? the grunt task already depends upon it

Copy link
Contributor Author

Choose a reason for hiding this comment

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

good point. I was messing around manually testing this without grunt at first. I'll remove it

@tbosch
Copy link
Contributor

tbosch commented Jan 30, 2014

Maybe merge commits and change 'WIP ...' messages?
LGTM otherwise.

@IgorMinar IgorMinar closed this in 7f4edaf Jan 30, 2014
@IgorMinar
Copy link
Contributor Author

fixed all the stuff and merged. thanks for reviews

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants