Basic bikeshed conversion #27

Merged
merged 1 commit into from Jan 20, 2016

3 participants

@metromoxie

Hi @devd, @fmarier, and @mozfreddyb. Per #12, this (finally) is the pull request that does a "basic" conversion of the spec to Bikeshed, without any additional changes. There are some basic formatting changes that are more or less required by Bikeshed, but the content should be (nearly?) identical. Let me know how it looks to y'all.

@devd

hmm .. do we really want to merge this now or wait till we hit REC and then switch to bikeshed for v2?

@metromoxie

Given that the Bikeshed conversion is in a separate file from the main spec, I'd prefer to commit the conversion now and just keep both specs in sync. That way we can all start taking a look at it. When we've moved to full spec, then we can replace spec.markdown with index.bikeshed.bs.

@mozfreddyb

I'm OK with keeping both files in parallel, assuming that we will do no (or really only minor) edits to v1 ( the markdown/respec version).
Then, all future work will only happen in the bikeshed file.

@metromoxie

Yes, what @mozfreddyb said :-) Make sure to still review it, though, to make sure it's a "just formatting" change that you don't feel like anything too major is being added.

@mozfreddyb

I'm currently reviewing and will issue pull requests for your branch, so you can take them one-by-one. Doing this will update this PR.

(We can squash all my commits into yours, I don't mind :))

@metromoxie

@mozfreddyb, given that you issued one pull request, which I've merged, can I take it that you approve of this CL otherwise?

@mozfreddyb

Yeah, let's ship it!

@metromoxie

Okay, I'll squash the commits and merge since I haven't heard any objections from @devd or @fmarier. Obviously we can always fix it as necessary :-)

@metromoxie metromoxie Convert to Bikeshed. Should be all formatting changes, which includes…
… plus some

bibliographic additions for Bikeshed references.
4b6816d
@metromoxie metromoxie merged commit d5b1943 into w3c:master Jan 20, 2016

1 check passed

Details continuous-integration/travis-ci/pr The Travis CI build passed
@metromoxie metromoxie deleted the metromoxie:bikeshed-conversion-basic branch Jan 20, 2016
This was referenced Mar 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment