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

Update the base generator to use the lib TwigGenerator #33

Closed
cedriclombardot opened this Issue Oct 26, 2011 · 14 comments

Comments

Projects
None yet
3 participants
@cedriclombardot
Member

cedriclombardot commented Oct 26, 2011

Just to add it in my todo to integrate the library what i just externalise https://github.com/cedriclombardot/TwigGenerator

@ghost ghost assigned cedriclombardot Oct 26, 2011

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot

cedriclombardot Nov 9, 2011

Member

In fact not, i'll mantain 2 codes to make you one deps less

Member

cedriclombardot commented Nov 9, 2011

In fact not, i'll mantain 2 codes to make you one deps less

@fzaninotto

This comment has been minimized.

Show comment
Hide comment
@fzaninotto

fzaninotto Nov 9, 2011

really? Why bother?

fzaninotto commented Nov 9, 2011

really? Why bother?

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot

cedriclombardot Nov 9, 2011

Member

Because it's really long to setup this bundle with all his vendors and i thinks to add one deps more for 2 files will be really boring for final users

Member

cedriclombardot commented Nov 9, 2011

Because it's really long to setup this bundle with all his vendors and i thinks to add one deps more for 2 files will be really boring for final users

@fzaninotto

This comment has been minimized.

Show comment
Hide comment
@fzaninotto

fzaninotto Nov 9, 2011

composer to the rescue!

fzaninotto commented Nov 9, 2011

composer to the rescue!

@willdurand

This comment has been minimized.

Show comment
Hide comment
@willdurand

willdurand Nov 9, 2011

Contributor

I don't think you're right Cédric. It's better to not duplicate code, and one more dep isn't really boring.

As @fzaninotto said, composer should help you. For now, try to write a shell script like the bin/vendors bundled with symfony-standard, or to write a short doc to setup the bundle faster than right now.

Contributor

willdurand commented Nov 9, 2011

I don't think you're right Cédric. It's better to not duplicate code, and one more dep isn't really boring.

As @fzaninotto said, composer should help you. For now, try to write a shell script like the bin/vendors bundled with symfony-standard, or to write a short doc to setup the bundle faster than right now.

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot

cedriclombardot Nov 9, 2011

Member

OK so i had it again to my todo

Member

cedriclombardot commented Nov 9, 2011

OK so i had it again to my todo

@willdurand

This comment has been minimized.

Show comment
Hide comment
@willdurand

willdurand Nov 9, 2011

Contributor

You may add an issue for your todo list, or will you add this on the roadmap page on the doc ?

Contributor

willdurand commented Nov 9, 2011

You may add an issue for your todo list, or will you add this on the roadmap page on the doc ?

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot
Member

cedriclombardot commented Nov 9, 2011

cedriclombardot added a commit that referenced this issue Nov 13, 2011

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot

cedriclombardot Nov 13, 2011

Member

FYI the setup/upgrade command is ready : php app/console admin:setup no i'll add the TwigGenerator vendor

Member

cedriclombardot commented Nov 13, 2011

FYI the setup/upgrade command is ready : php app/console admin:setup no i'll add the TwigGenerator vendor

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot
Member

cedriclombardot commented Nov 13, 2011

Done

@fzaninotto

This comment has been minimized.

Show comment
Hide comment
@fzaninotto

fzaninotto commented Nov 13, 2011

awesome

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot

cedriclombardot Nov 24, 2011

Member

The screencast for the command line admin:setup is here : http://youtu.be/c5Q2v8llnNU?hd=1 (see in full screen & hd for better results), next one the creation of the generator.yml

Member

cedriclombardot commented Nov 24, 2011

The screencast for the command line admin:setup is here : http://youtu.be/c5Q2v8llnNU?hd=1 (see in full screen & hd for better results), next one the creation of the generator.yml

@fzaninotto

This comment has been minimized.

Show comment
Hide comment
@fzaninotto

fzaninotto Nov 24, 2011

Great. I suggest that you use bigger fonts for the next screencast, and
probably a smaller screen size as well.

Also, using VI kind of hides when you switch files. I's hard to follow. If
you could use a tree plugin showing the file structure, which file you work
on, that would help.

Apart from that, very dramatic music!

2011/11/24 cedric lombardot <
reply@reply.github.com

The screencast for the command line admin:setup is here :
http://youtu.be/c5Q2v8llnNU?hd=1 (see in full screen & hd for better
results), next one the creation of the generator.yml


Reply to this email directly or view it on GitHub:

https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/33#issuecomment-2857063

fzaninotto commented Nov 24, 2011

Great. I suggest that you use bigger fonts for the next screencast, and
probably a smaller screen size as well.

Also, using VI kind of hides when you switch files. I's hard to follow. If
you could use a tree plugin showing the file structure, which file you work
on, that would help.

Apart from that, very dramatic music!

2011/11/24 cedric lombardot <
reply@reply.github.com

The screencast for the command line admin:setup is here :
http://youtu.be/c5Q2v8llnNU?hd=1 (see in full screen & hd for better
results), next one the creation of the generator.yml


Reply to this email directly or view it on GitHub:

https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/33#issuecomment-2857063

@cedriclombardot

This comment has been minimized.

Show comment
Hide comment
@cedriclombardot

cedriclombardot Nov 24, 2011

Member

I see the problem on fonts when i ve upload the video on YouTube my
rsolution was bigguest and i register on 22 pouces screen. The next will
ve better

And about bon its because my computer can t run clipse and capture on same
time it sas already very slow. Probably change computer or use nerd tree in
vim next time.

Thanks for feedback
Le 24 nov. 2011 09:04, "Francois Zaninotto" <
reply@reply.github.com>
a crit :

Great. I suggest that you use bigger fonts for the next screencast, and
probably a smaller screen size as well.

Also, using VI kind of hides when you switch files. I's hard to follow. If
you could use a tree plugin showing the file structure, which file you work
on, that would help.

Apart from that, very dramatic music!

2011/11/24 cedric lombardot <
reply@reply.github.com

The screencast for the command line admin:setup is here :
http://youtu.be/c5Q2v8llnNU?hd=1 (see in full screen & hd for better
results), next one the creation of the generator.yml


Reply to this email directly or view it on GitHub:

https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/33#issuecomment-2857063


Reply to this email directly or view it on GitHub:

https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/33#issuecomment-2859231

Member

cedriclombardot commented Nov 24, 2011

I see the problem on fonts when i ve upload the video on YouTube my
rsolution was bigguest and i register on 22 pouces screen. The next will
ve better

And about bon its because my computer can t run clipse and capture on same
time it sas already very slow. Probably change computer or use nerd tree in
vim next time.

Thanks for feedback
Le 24 nov. 2011 09:04, "Francois Zaninotto" <
reply@reply.github.com>
a crit :

Great. I suggest that you use bigger fonts for the next screencast, and
probably a smaller screen size as well.

Also, using VI kind of hides when you switch files. I's hard to follow. If
you could use a tree plugin showing the file structure, which file you work
on, that would help.

Apart from that, very dramatic music!

2011/11/24 cedric lombardot <
reply@reply.github.com

The screencast for the command line admin:setup is here :
http://youtu.be/c5Q2v8llnNU?hd=1 (see in full screen & hd for better
results), next one the creation of the generator.yml


Reply to this email directly or view it on GitHub:

https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/33#issuecomment-2857063


Reply to this email directly or view it on GitHub:

https://github.com/cedriclombardot/AdmingeneratorGeneratorBundle/issues/33#issuecomment-2859231

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment