Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(.travis): enable auto-publish #129

Merged
merged 10 commits into from
May 23, 2019
Merged

Conversation

marcoscaceres
Copy link
Member

@marcoscaceres marcoscaceres commented May 22, 2019

@marcoscaceres marcoscaceres requested a review from siusin May 22, 2019 01:16
@marcoscaceres
Copy link
Member Author

@siusin @deniak, having some problems with this that I can't figure out:
https://labs.w3.org/echidna/api/status?id=431c7058-5045-49d9-bd87-c418ec0eff2d

I think the "this-latest-shortname" might be invalid because BikeShed adds a "-1-" for the "level" (which is required by BikeShed), but I'm not sure. Could you confirm? We can ping Tab if true.

I'm not sure why the copyright is generating issues, however. The generated markup seems fine, unless there is something obvious I'm not seeing.

@deniak
Copy link
Member

deniak commented May 22, 2019

Here are the headers of the documents:

This version:
    https://www.w3.org/TR/2019/WD-fileapi-1-20190522/
Latest published version:
    https://www.w3.org/TR/FileAPI/
Editor's Draft:
    https://w3c.github.io/FileAPI/
Previous Versions:
    https://www.w3.org/TR/2017/WD-FileAPI-20171026/

The first thing I notice is that it'll be considered as a shortname change and Echidna currently doesn't support that so you would have to send a request to webreq@w3.org for that version at least.
Then, I'm not sure how bikeshed generates the document but the shortnames in the "This version" link and in the "Latest published version" should be the same. So in this case, you should have something like

This version:
    https://www.w3.org/TR/2019/WD-fileapi-1-20190522/
Latest published version:
    https://www.w3.org/TR/fileapi-1/

Regarding the copyright, the link for MIT, ERCIM and Keio should be under https.

@marcoscaceres
Copy link
Member Author

Thanks @deniak! I'll send a fix for those HTTP URLs and figured out how to fix the levels thing.

@marcoscaceres
Copy link
Member Author

@mkruisselbrink, just confirming that you are ok with switching off the levels (see above, as I don't think we want to request a new short name).

I've fixed the https issues over on BikeShed speced/bikeshed#1478

@mkruisselbrink
Copy link
Collaborator

Yes, thanks. This looks good to me.

@marcoscaceres
Copy link
Member Author

Thanks @mkruisselbrink for confirming. Just ironing out last little things.

@marcoscaceres
Copy link
Member Author

Blocked on speced/bikeshed#1480

@marcoscaceres marcoscaceres merged commit e205358 into master May 23, 2019
@marcoscaceres marcoscaceres deleted the marcoscaceres-patch-2 branch May 23, 2019 22:27
@marcoscaceres
Copy link
Member Author

@deniak, final thing is the switching deliverer issue. How do we proceed with that? Anything I can do to help? It’s currently blocking publication of all other specs in the wg.

@deniak
Copy link
Member

deniak commented May 24, 2019

@marcoscaceres unfortunately, Echidna is not able yet to handle change of deliverer as it requires some IPP changes.
You will need to send a publication request to webreq@w3.org at least for that version. Once it has been published, you will be able to publish the next versions with echidna.

@siusin
Copy link
Contributor

siusin commented May 24, 2019

@deniak Ok, I'll do that.

Do we need to republish all the WebPlatform specs under WebApps WG?

@deniak
Copy link
Member

deniak commented May 24, 2019

yeah, Echidna will reject your document if it changes deliverers

@siusin
Copy link
Contributor

siusin commented May 24, 2019

All right, in that case, I'll send a pubreq that includes all the specs next Monday :)

@marcoscaceres
Copy link
Member Author

Filed w3c/webappswg#7 about republishing ... will file bugs for the other specs.

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.

None yet

4 participants