- From: Tobie Langel <notifications@github.com>
- Date: Wed, 07 Sep 2016 04:42:08 -0700
- To: heycam/webidl <webidl@noreply.github.com>
- Message-ID: <heycam/webidl/pull/158/c245255071@github.com>
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