Skip to content

[web-animations-1] Publish new Working Draft #6211

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

Closed
birtles opened this issue Apr 14, 2021 · 12 comments
Closed

[web-animations-1] Publish new Working Draft #6211

birtles opened this issue Apr 14, 2021 · 12 comments

Comments

@birtles
Copy link
Contributor

birtles commented Apr 14, 2021

Hi,

I would like to publish a new working draft of Web Animations level 1.

It has been over 2 years since the last working draft was published. In that time the following changes have been made:

As far as I know, all these changes have been implemented in all three major browser engines.

With regards to progressing the spec to REC, the last major issue is #5394 which all the major browser vendors have agreed upon. I have started on the spec/test/implementation work but it will take more time. After that, the remaining issues are hopefully minor.

I don't expect to be on the call unfortunately.

@css-meeting-bot
Copy link
Member

The CSS Working Group just discussed [web-animations-1] Publish new Working Draft, and agreed to the following:

  • RESOLVED: Publish a new WD of web-animations
The full IRC log of that discussion <dael> Topic: [web-animations-1] Publish new Working Draft
<dael> github: https://github.com//issues/6211
<dael> astearns: Anyone have concerns?
<dael> astearns: Objections?
<dael> RESOLVED: Publish a new WD of web-animations
<dael> fantasai: Can we disable test harness, publish, and repub when we figure this out?
<dael> plinss: Shoudln't be an issue. 1 line of script
<dael> plinss: Probably auto-gen by bikeshed
<dael> TabAtkins: Yeah, part of boilerplate. Can remove temp.
<dael> astearns: Not bikeshed flag?
<dael> TabAtkins: No, can kill it from boilerplate
<dael> astearns: Sounds good. Can you add comment to the issue reminding us to put it back in?

@birtles
Copy link
Contributor Author

birtles commented Apr 22, 2021

@fantasai I was wondering what "this" is from your comment:

fantasai: Can we disable test harness, publish, and repub when we figure this out?

Is there something I need to disable before publishing the WD?

@tabatkins
Copy link
Member

If you're on most current main of Bikeshed, the testharness annotations are disabled for CSSWG specs currently, so it'll work.

@birtles
Copy link
Contributor Author

birtles commented Apr 22, 2021

Thanks! I recently switched to installing bikeshed through pip3 rather than a local checkout so I guess I'm not.
(I've updated it a few hours ago but bikeshed --version just returns ???.)

@birtles
Copy link
Contributor Author

birtles commented May 7, 2021

I finally got a trunk version of bikeshed running. I fixed the dark mode colors from 9586314 too since the validator didn't like rgba() without commas.

However, now I get stuck with The document could not be published: Cannot read property 'message' of undefined

Log here: https://labs.w3.org/echidna/api/status?id=66c32572-9e9a-4583-b819-0543b13837f7

@fantasai Any idea who I should ask?

@tabatkins
Copy link
Member

Ooof, that looks like a bug in the publication code. @plehegar?

@svgeesus
Copy link
Contributor

It does; especially because Specberus (pubrules checker) was already passed

time "2021-05-07T00:12:34.731Z"
fact "The file has been retrieved."
1  
time "2021-05-07T00:12:40.209Z"
fact "The metadata have been extracted."
2  
time "2021-05-07T00:12:51.650Z"
fact "The document passed Specberus."
3  
time "2021-05-07T00:12:51.653Z"
fact "The document passed the transition checker."

@svgeesus
Copy link
Contributor

@birtles if it is still not working by Wednesday let me know and I will publish it manually. (Or try to, since we don't know what triggers the error)

@birtles
Copy link
Contributor Author

birtles commented May 11, 2021

@svgeesus Thanks! I just retried now and I get the same error (log here).

(Note to self: When publishing in the morning in Japan, you need to add Date: <yesterday's date in yyyy-mm-dd format> or else Specberus complains, "The publication date of this document must be set to today".)

@svgeesus
Copy link
Contributor

Hmm

[EXCEPTION] The document could not be parsed, it's neither a TR document nor a Member Submission.
[EXCEPTION] The document date could not be parsed.

Pubrules checker is unhappy :(

@svgeesus
Copy link
Contributor

w3c/specberus#1234

@birtles
Copy link
Contributor Author

birtles commented May 18, 2021

New WD is published: https://www.w3.org/TR/2021/WD-web-animations-1-20210518/
ED has been updated accordingly (or at least it will be once CI finishes running).

@birtles birtles closed this as completed May 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants