Sign up Process Improvements about confide

[andrew13]

Yep, fork > code changes > run and update unit tests > create pull request

Thanks for contributing back to the project.

[christoferd]

Hey Andrew,

Sorry to bother you with this. I got this far, "Yep, fork > code changes"

Now I don't get how to keep my application files intact, merge your new
updates, but share only certain files back to you without giving you
private code. :-/

And, what if modifications are in "confide"? I think that should be a
separate fork - yes? (I've noticed /vendor/ is being ignored in
Starter-Site) .... I'm guessing that the best move is to just fork confide
next, code changes, push to fork, then send pull request. That will any
issues with my own app code, right?

Cheers,

Chris De David
Phone: [phone] | Mobile: [phone] | Skype:
chris.dedavid
Mindflow.com.au

On 14 May 2013 11:45, Andrew [email protected] wrote:

Yep, fork > code changes > run and update unit tests > create pull request

Thanks for contributing back to the project.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/71#issuecomment-17852276
.

[andrew13]

Confide is a separate project. You would need to clone that separately as well.

I would do development in a clean repo and not in your own application directory.

One way to do it is to add a different repo to the confide directory.
You'll want to cd in to /vendors/zizaco/confide
then you'll need to add a new git remote

git remote add myconfide [email protected]:MindflowAU/confide.git

Then make your changes and then push to myconfide instead of origin.

git push myconfide master

[christoferd]

Ok, thanks.

Cheers,

Chris De David
Phone: [phone] | Mobile: [phone] | Skype:
chris.dedavid
Mindflow.com.au

On 17 May 2013 04:15, Andrew [email protected] wrote:

Confide is a separate project. You would need to clone that separately as
well.

I would do development in a clean repo and not in your own application
directory.

—
Reply to this email directly or view it on GitHubhttps://github.com//issues/71#issuecomment-18018985
.

[Zizaco]

@MindflowAU, I'm closing this issue because as soon as you send a pull request, an new issue will be created with the code that you want to merge.