cfjedimaster / brackets-jshint

Adds JSHint support to Brackets
MIT License
131 stars 41 forks source link

A few cleanup fixes #7

Closed WebsiteDeveloper closed 11 years ago

WebsiteDeveloper commented 11 years ago

4

cfjedimaster commented 11 years ago

FYI, I found some major issues with the extension. I'm doing some low level plumbing and may reject your patch so you can re-do it on the latest. (Or I may merge by hand.) Just as an FYI in case you are seeing bad results.

On Tue, Jun 4, 2013 at 12:06 PM, Bernhard Sirlinger < notifications@github.com> wrote:

4 https://github.com/cfjedimaster/brackets-jshint/issues/4

You can merge this Pull Request by running

git pull https://github.com/WebsiteDeveloper/brackets-jshint Improvements

Or view, comment on, or merge it at:

https://github.com/cfjedimaster/brackets-jshint/pull/7 Commit Summary

  • Make use of Resizer constants when avaliable
  • Add new bootstrap classes

File Changes

  • M main.jshttps://github.com/cfjedimaster/brackets-jshint/pull/7/files#diff-0(58)

Patch Links:

Raymond Camden, Adobe Developer Evangelist

Email : raymondcamden@gmail.com Blog : www.raymondcamden.com Twitter: cfjedimaster

WebsiteDeveloper commented 11 years ago

yeah no problem

cfjedimaster commented 11 years ago

Going to go ahead and close this as I know what mod was already done. I'm holding off doing the bootstrap changing till 26 is really released.