Skip to content
This repository has been archived by the owner on Feb 17, 2021. It is now read-only.

Reorganize repo and automate release process #61

Merged
merged 10 commits into from
Mar 11, 2014

Conversation

kate2753
Copy link
Contributor

@kate2753 kate2753 commented Mar 4, 2014

This pull request includes repo restructure to clearly separate source files from distribution files. It also includes changes to grunt build to automate release process.

See more details in #59

@@ -19,8 +19,13 @@ CSS is compiled using [LESS](http://lesscss.org/).

Compiling
=========
Hopscotch uses Grunt for minification and hinting. Make sure you have [`npm`](https://npmjs.org) installed. Run `npm install` from the hopscotch directory to load in the dev dependencies, then run `grunt`.
Hopscotch uses Grunt for build and testing. You need to have [`npm`](https://npmjs.org) and [grun cli](http://gruntjs.com/getting-started) installed.
Copy link
Collaborator

Choose a reason for hiding this comment

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

typo (grun -> grunt)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

doh! Nice catch. Will fix

@timlindvall
Copy link
Collaborator

Added a couple of nitpicks in the diff, but all in all the new directory structure and gruntfile looks good to me!

@eoneill
Copy link
Contributor

eoneill commented Mar 10, 2014

This looks good to merge to me. If there's no objection by EOD, I'm going to merge this.

eoneill added a commit that referenced this pull request Mar 11, 2014
restructure repo for maintainability + grunt tasks for releases
@eoneill eoneill merged commit 61b03dc into LinkedInAttic:master Mar 11, 2014
@kate2753 kate2753 deleted the gruntRelease branch March 11, 2014 18:38
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