Re: [heycam/webidl] Bikeshed conversion (#158)

Rewrote history in a bunch of different ways to compare and pushed them to separate branches.

### A single commit ###

Branch: https://github.com/tobie/webidl/commits/bs-single-commit

This is best avoided as it severs the connection between index.xml and index.bs.

### Two commits ###

Branch: https://github.com/tobie/webidl/commits/bs-2-commits

Splits the work up into:

1. filename change
2. everything else 

This keeps the connection between index.xml and index.bs and centralizes all the conversion-related blame in a single commit. However, it looses the integration and removal of [webidl2bikeshed](https://github.com/tobie/webidl2bikeshed) as a submodule.


### Three commits ###

Branch: https://github.com/tobie/webidl/commits/bs-3-commits

Splits the work up into:

1. filename change
2. automated changes & manual fixes
3. repo cleanup

This keeps:

- the connection between index.xml and index.bs,
- the usage and removal of the  [webidl2bikeshed](https://github.com/tobie/webidl2bikeshed) submodule (not sure how important that is, probably not really),
- the conversion git blame on a single commit.

### Four commits ###

Branch: https://github.com/tobie/webidl/commits/bs-4-commits

Splits the work up into:

1. filename change
2. automated changes
3. repo cleanup
4. manual fixes 

This keeps the connection between index.xml and index.bs and the usage and removal of the  [webidl2bikeshed](https://github.com/tobie/webidl2bikeshed) submodule.

The conversion git blame now becomes scattered on two commits: the automation one and the manual edit one. That's a lot more messy, imho.

### Conclusion ###

Given the above, I'd go for the [two](https://github.com/tobie/webidl/commits/bs-2-commits)- or [three](https://github.com/tobie/webidl/commits/bs-3-commits)- commits option.


-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/heycam/webidl/pull/158#issuecomment-245255071

Received on Wednesday, 7 September 2016 11:42:38 UTC