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

easing-tests.js is out of date #15483

Closed
egirard opened this issue Feb 20, 2019 · 6 comments
Closed

easing-tests.js is out of date #15483

egirard opened this issue Feb 20, 2019 · 6 comments

Comments

@egirard
Copy link

egirard commented Feb 20, 2019

Looking at easing-tests.js in wpt, moz, and chromium, it seems that mozilla's is more recent than the others. It seems to have been updated in https://phabricator.services.mozilla.com/D9309 but those changes weren't upstreamed.

@birtles, @BorisChiou can you advise?

@egirard
Copy link
Author

egirard commented Feb 20, 2019

More specifically, there's a section starting at line 44 that includes:

 {
   desc: 'frames function',
   easing: 'frames(5)',
   easingFunction: framesTiming(5)
 },

@BorisChiou
Copy link
Member

BorisChiou commented Feb 20, 2019

It seems the wpt bot didn't upstream the update of easing-tests.js. I can create a PR to update it if needed.

@BorisChiou
Copy link
Member

@jgraham do you have any idea to re-trigger the wpt bot for the patch (https://phabricator.services.mozilla.com/D9309)?

@birtles
Copy link
Contributor

birtles commented Feb 21, 2019

Yeah, it looks like the sync bot picked up the changes in parts 2 and 8 of that patch series but not part 1:

https://github.com/web-platform-tests/wpt/pull/13868/commits

There may have been a merge conflict apparently:

https://bugzilla.mozilla.org/show_bug.cgi?id=1502788#c1

@birtles
Copy link
Contributor

birtles commented Feb 26, 2019

I followed up with @jgraham on this. He suspects a misresolved merge conflict and is going to look into it.

@birtles
Copy link
Contributor

birtles commented Mar 4, 2019

Fixed by a4bfa25.

@birtles birtles closed this as completed Mar 4, 2019
birtles added a commit to birtles/web-platform-tests that referenced this issue May 9, 2019
As per web-platform-tests#15483, there was some sort of merge conflict that meant that
part 1 of [Mozilla bug
1496619](https://bugzilla.mozilla.org/show_bug.cgi?id=1496619) did not
get properly merged to web-platform-tests.

PR web-platform-tests#15587 attempted to fix this but failed to delete two files in
css-timing that were deleted in this changeset:

  https://hg.mozilla.org/mozilla-central/rev/b46aa2883e42
jdm pushed a commit that referenced this issue May 9, 2019
As per #15483, there was some sort of merge conflict that meant that
part 1 of [Mozilla bug
1496619](https://bugzilla.mozilla.org/show_bug.cgi?id=1496619) did not
get properly merged to web-platform-tests.

PR #15587 attempted to fix this but failed to delete two files in
css-timing that were deleted in this changeset:

  https://hg.mozilla.org/mozilla-central/rev/b46aa2883e42
marcoscaceres pushed a commit that referenced this issue Jul 23, 2019
As per #15483, there was some sort of merge conflict that meant that
part 1 of [Mozilla bug
1496619](https://bugzilla.mozilla.org/show_bug.cgi?id=1496619) did not
get properly merged to web-platform-tests.

PR #15587 attempted to fix this but failed to delete two files in
css-timing that were deleted in this changeset:

  https://hg.mozilla.org/mozilla-central/rev/b46aa2883e42
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