search

sysapps / runtime

Runtime and Security Model for Hosted Web Applications and Packaged Applications
6 stars 13 forks source link

Fix some small typos in the index.html file. #5

Closed jplyle closed 11 years ago

jplyle commented 11 years ago

Hi Mounir / Ming Jin,

I don't know whether any changes are possible during the current phase of the specification process, but I spotted a few typos and thought you might appreciated them being fixed in the FPWD.

Thanks,

John

ming-jin commented 11 years ago

+1

marcoscaceres commented 11 years ago

These all look good to me. We should merge these in ASAP.

mounirlamouri commented 11 years ago

Could you by any chance update your pull request? I unfortunately can't merge it as-is.

mounirlamouri commented 11 years ago

I just merged a few PR from Christophe regarding typos so I guess part of this patch is no longer needed.

jplyle commented 11 years ago

Sure, you're the editor - if you think all the typos are addressed, that's fine. I haven't gone through to check.

marcoscaceres commented 11 years ago

@jplyle be good to merge the upstream and check. That way we can still merge this and close.

mounirlamouri commented 11 years ago

I would bet some fixes from that patch still applies but the patch itself no longer applies so it needs to be updated so I can merge it.

jplyle commented 11 years ago

Ok, I'll do that when I get a chance... I'm on a different machine right now, but I'll try and do it today.

jplyle commented 11 years ago

Whitespace and merge conflicts (along with my ineptitude with a different merge tool) appear to have made a mess of this. I'll have another go.

jplyle commented 11 years ago

I'd like to close this PR and use the alternative I've just issued - https://github.com/sysapps/runtime/pull/40 - thanks.

mounirlamouri commented 11 years ago

Just merged PR #40 so closing this.