Skip to content

WIP: Squash all commits cloning method#65

Open
dhimmel wants to merge 1 commit intomanubot:masterfrom
dhimmel:squash-all
Open

WIP: Squash all commits cloning method#65
dhimmel wants to merge 1 commit intomanubot:masterfrom
dhimmel:squash-all

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