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.

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

WIP: Squash all commits cloning method #65

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dhimmel
Copy link
Member

@dhimmel dhimmel commented Aug 31, 2017

[DO NOT MERGE]

This is an experimental PR to investigate a "sqaush all" method for cloning this repo. When using this method, all manubot-rootstock commits up to the cloning moment are squashed. The result is like dhimmel/rephetio-manuscript@4ae7a48.

The benefit of this method is that the commit history of cloned manuscripts is less cluttered.

Still looking into how merges will work... presumably we want to squash those as well.

Squash all method for merging updates not yet implemented
@rgieseke
Copy link
Contributor

rgieseke commented Oct 9, 2018

Is there an easy way to make an update for repos created this way? Just download the zip and overwrite files?

@dhimmel
Copy link
Member Author

dhimmel commented Oct 17, 2018

Is there an easy way to make an update for repos created this way?

I think I played around with this once, but didn't immediately find a perfect solution. However, I'm thinking the best solution would be to cherry pick the desired range of commits (i.e. commits not yet included in your manuscript instance). After you finish the cherry picking, dealing with any issues, then perhaps you could squash all the newly added commits.

Just download the zip and overwrite files?

This isn't ideal as it'll clobber any changes you've made to these files. Perhaps however, you could do this and then do git add --patch and select which part of the diff to commit and preserve. Afterwards, you could reset to get rid of the uncommitted changes.

@rgieseke
Copy link
Contributor

Thanks @dhimmel - I went with manually copying over and git add --patch :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants