Skip to content
This repository has been archived by the owner on Oct 30, 2018. It is now read-only.

archetype and build type for crt/updater/ide #643

Merged
merged 24 commits into from
Oct 22, 2012

Conversation

isao
Copy link
Contributor

@isao isao commented Oct 18, 2012

new archetype: mojito create app hybrid myapp
new build type: mojito build hybrid path/to/packages/dir
https://gist.github.com/26623632f9c593be9078
though the build command was split into seven different sub modules, the eloc count is lower at the moment.

new support for user-specified archetypes: mojito create custom path/to/your/archetype mything

@drewfish
Copy link
Contributor

For finding custom archetypes, I would have preferred if it used the resource store (store.getResources('server', {}, {type:'archetype', subtype:archetypeType})), so that archetypes delivered by NPM modules could be easily used.

create-mojit-default one per orig docs
@isao
Copy link
Contributor Author

isao commented Oct 22, 2012

@drewfish We discussed this when I tried implementing the hybrid archetype as a package-- RS doesn't support it, and the existing create.js doesn't use mojito or the resource store at all yet.

This feature is pretty minor in terms of code changes, and may be useful for the IDE/CRT integration.

I agree we should support of npm packaging of archetypes in the future, as planned, so that archetypes can be sanely shared, versioned, etc. If that future/planned feature blocks this entire PR, let me know and I'll make a new one without the mojito create custom feature.

@drewfish
Copy link
Contributor

OK cool. I don't imagine that anything in this PR will block future use of archetypes delivered via NPM modules.

So, +1.

isao pushed a commit that referenced this pull request Oct 22, 2012
new archetype: mojito create app hybrid myapp
new build type: mojito build hybrid path/to/packages/dir
new custom archetypes: mojito create custom path/to/your/archetype mything
@isao isao merged commit a87ac15 into YahooArchive:develop Oct 22, 2012
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.

2 participants