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

Use new IDL mixin syntax in cssom.idl and cssom-view.idl #8772

Closed
wants to merge 1 commit into from
Closed

Use new IDL mixin syntax in cssom.idl and cssom-view.idl #8772

wants to merge 1 commit into from

Conversation

romandev
Copy link
Member

@romandev romandev commented Dec 21, 2017

@ghost
Copy link

ghost commented Dec 21, 2017

Build PASSED

Started: 2017-12-21 07:56:49
Finished: 2017-12-21 08:07:09

This report has been truncated because the number of unstable tests exceeds GitHub.com's character limit for comments (65536 characters).

Failing Jobs

  • MicrosoftEdge:14.14393

Unstable Browsers

Browser: "Microsoftedge 14.14393" (failures allowed)

View in: WPT PR Status | TravisCI

@foolip
Copy link
Member

foolip commented Jan 14, 2018

@romandev, can you rebase this so that we get a new Travis run? https://pulls.web-platform-tests.org/pull/8772 doesn't have any results (@mattl FYI, maybe because of downtime?)

This should be merged together with w3c/csswg-drafts#2123, right?

(This is an open PR that would require changes if I do #9026, that's why I'm here.)

@ghost
Copy link

ghost commented Jan 15, 2018

Build PASSED

Started: 2018-01-15 16:55:20
Finished: 2018-01-15 17:02:21

This report has been truncated because the number of unstable tests exceeds GitHub.com's character limit for comments (65536 characters).

Failing Jobs

  • safari:11.0
  • MicrosoftEdge:14.14393

Unstable Browsers

Browser: "Microsoftedge 14.14393" (failures allowed)

View in: WPT PR Status | TravisCI

@romandev
Copy link
Member Author

@foolip Rebased this patch.

This should be merged together with w3c/csswg-drafts#2123, right?

Yep, right.

Thanks!

@domenic domenic removed their request for review April 20, 2018 21:15
@foolip
Copy link
Member

foolip commented Aug 15, 2018

This was fixed in #9777 and #11819.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants