Skip to content

Re #46: Restructures the project to provide a working demo#47

Merged
KidSysco merged 11 commits into
KidSysco:masterfrom
benjamin-albert:demo
Feb 15, 2016
Merged

Re #46: Restructures the project to provide a working demo#47
KidSysco merged 11 commits into
KidSysco:masterfrom
benjamin-albert:demo

Conversation

@benjamin-albert
Copy link
Copy Markdown
Collaborator

As we've discussed earlier in #46 and on Slack, this pull moves the gh-pages branch content into the /demo directory.

It also sets up a grunt task to automatically merge the contents of the /demo directory with the gh-pages branch and push the gh-pages branch to origin (aka GitHub).

The gh-pages task needs to be run separately by calling:

grunt gh-pages

IMPORTANT: I've made some change to the demo files so you need to run gh-pages task for them to be visible.

I wanted the minified output to come off of github.io servers and to provide a good simple example for installing the plugin in index.html.

This required me to:

  • Move the minified output into the /demo directory.
  • Make grunt stop generating (and delete) demo.min.js and demo.min.css.
  • Move the source JavaScript and CSS into the /src directory (because I don't want people to see the source and say ohh there's not minded version).

IMPORTANT: The new project structure requires us to point the demo on JSFiddle to the new files in the demo directory.

In addition I:

  • Made jquery.maskedinput.js in the /test directory come from bower instead of being in the repository.
  • Fixed the Error: No tests specified message in the Travis-CI build by adding an npm test command that just calls grunt test.

@benjamin-albert benjamin-albert changed the title Re #46: Restructures the reject to provide a working demo Re #46: Restructures the project to provide a working demo Feb 12, 2016
@benjamin-albert
Copy link
Copy Markdown
Collaborator Author

Perhaps we should consider renaming the /demo directory to something that makes it more clear that the minified production files are in there.

I find the name /dist used quite often.

KidSysco added a commit that referenced this pull request Feb 15, 2016
Re #46: Restructures the project to provide a working demo
@KidSysco KidSysco merged commit b763f80 into KidSysco:master Feb 15, 2016
@benjamin-albert benjamin-albert deleted the demo branch February 16, 2016 18:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants