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.

Sign up for GitHub

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

Jump to bottom

Codegen templates #6

Open
scottux opened this issue Sep 29, 2013 · 3 comments
Open

Codegen templates #6

scottux opened this issue Sep 29, 2013 · 3 comments

Comments

@scottux
Copy link
Member

scottux commented Sep 29, 2013

When we generate code from templates, we need to be able to specify an ORM class to extend. This may involve creating mirificus-orm or mirificus-model
@scottux
Copy link
Member Author

scottux commented Sep 29, 2013

How many templates should we bundle and how do we deal with loading custom ones?

@var-foo
Copy link
Member

var-foo commented Sep 30, 2013

We need to determine which templates will be the most commonly use and bundle those. As for loading custom ones... idk yet. Lemme think on that.

var-foo pushed a commit that referenced this issue Sep 30, 2013
Start cleanup of codegen templates addresses #6
4345f1c 
Change all raw output lines to sprintf so that everything lives inside
php tags. This makes everything a bit easier to digest.
@scottux
Copy link
Member Author

scottux commented Oct 1, 2013

We could just have a "base Model" setting and use that to generate the ORM parent.

var-foo added a commit to var-foo/mirificus-codegen that referenced this issue Oct 10, 2013
@var-foo
Revert "Start cleanup of codegen templates addresses SensibleUX#6"
de96aa2 
This reverts commit 4345f1c.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@scottux @var-foo