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.

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

WIP - Make 'dist' path configurable via option #16

Merged
merged 2 commits into from
Mar 12, 2014

Conversation

tomfulton
Copy link
Collaborator

This mimics the grunt watch:dev --target="D:\MySite" option from Archetype, but with a simpler approach (thanks @taylorsmith)

Instead of adding new tasks, we're making the /dist/ directory configurable, so existing tasks can be used easily (ie, grunt or grunt watch)

Examples:

grunt --target="D:\MySite"

grunt watch --target="D:\MySite"

Note that we're now building using the full folder structure (App_Plugins/<name>/<files>) instead of a flat structure, which should make deployment a bit easier, especially when we add other folders like /bin/

Implements #6

Grunt will build the files to /dist/ by default, but you can now specify a different target to build to by passing the --target option:

Ex:
grunt --target="D:\MySite"
grunt watch --target="D:\MySite"

* Note that we're now building (to dist or wherever) using the full folder structure (App_Plugins/<name>) which should make deployment a bit easier, especially when we add other folders like /bin/
@tomfulton
Copy link
Collaborator Author

@taylorsmith This what you had in mind?

@warrenbuckley
Copy link
Owner

@tomfulton Brill merged in!

Will try & merge in with my target stuff shortly which checks target option exists & that the value passed in is a directory & that it exists

warrenbuckley added a commit that referenced this pull request Mar 12, 2014
WIP - Make 'dist' path configurable via option
@warrenbuckley warrenbuckley merged commit db0c113 into master Mar 12, 2014
@warrenbuckley warrenbuckley deleted the feature/deploy-target branch March 12, 2014 10:33
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.

None yet

2 participants