`amethyst new` doesn't put authors in Cargo.toml

[White-Oak]

It is probably not so easy, but I think it is possible to get git name and email on a system and add put them into author field of Cargo.toml.

[ebkalderon]

Note: This proposal will also solve issues #8 and #10.

We may be able to work around manual manipulation of the TOML text. I propose the following:

1. Only the src and resources directories remain in the "new_project.zip" file.
2. At runtime, amethyst new [name] calls cargo new [name]. A fresh Cargo project is created.
3. The contents of "new_project.zip" are unzipped into the newly-created folder.
4. The [dependencies] details are appended to the end of the "Cargo.toml" file.

Does that seem reasonable to you?

[ebkalderon]

Note: This proposal will also solve issues #8 and #10.

We may be able to work around manual manipulation of the TOML text. I propose the following:

1. Only the src and resources directories remain in the "new_project.zip" file.
2. At runtime, amethyst new [name] calls cargo new [name]. A fresh Cargo project is created.
3. The contents of "new_project.zip" are unzipped into the newly-created folder.
4. The [dependencies] details are appended to the end of the "Cargo.toml" file.

Does that seem reasonable to you?

[White-Oak]

Yes, it is a much better and easier approach!

[White-Oak]

Yes, it is a much better and easier approach!

[ebkalderon]

Cool! I can get started working on it a bit later today, though if you're impatient and would like to submit a pull request sooner, I welcome it. 😉

[ebkalderon]

Cool! I can get started working on it a bit later today, though if you're impatient and would like to submit a pull request sooner, I welcome it. 😉

[White-Oak]

@ebkalderon yeah, I would like to submit a pull request if you don't mind :)

[White-Oak]

@ebkalderon yeah, I would like to submit a pull request if you don't mind :)

[ebkalderon]

@White-Oak Please, go right ahead. 😄

[ebkalderon]

@White-Oak Please, go right ahead. 😄

[ebkalderon]

This has been fixed by pull request #12. Marking as resolved and closing.

[ebkalderon]

This has been fixed by pull request #12. Marking as resolved and closing.