-
Notifications
You must be signed in to change notification settings - Fork 334
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
Update dependencies #1070
Update dependencies #1070
Conversation
aee5a99
to
4215477
Compare
The new version of axe fails because we're using (what they might consider?) to be redundant roles, for example |
some of the prefixes, like i'm not sure about |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks like -webkit-flex
and -webkit-align-items
haven't been required since Safari 6 so it's odd that those are being added back in. Maybe something has changed with the new version of autoprefixer
. Interestingly, -webkit-justify-content
is required until Safari 10 so it seems sensible to have that added. I tested the footer in Safari 5 and 6 before and after changes from this PR and also can't see a difference.
Removing the -moz-
ones seems legitimate as -moz-column-gap
and -moz-column-count
haven't been required since Firefox 51. I think 51 has probably just dropped off the list of browsers we prefix for.
The rest of the PR looks great, thanks @NickColley (as long as we're okay to have those Safari 6 prefixes added to the next package
release - I don't think it should be a blocker).
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blocking as this bumps event-stream to 3.3.6 and introduces a dependency on flatmap-stream, which is believed to be a compromised package.
nodemon 1.18.7 is out that removes the affected dependencies https://github.com/remy/nodemon/releases/tag/v1.18.7 |
9826be9
to
a66d9f6
Compare
This is required for IE8 support
Getting some false positive failures with version 3 of axe-core that'll need to be investigated on it's own.
Since the package-lock structure has changed this test is failing
6c180a8
to
0d7ec1e
Compare
The tests appear to be failing because Other than that, I'm happy with this. |
5d4f616
to
b40bf50
Compare
After updating these dependencies and running
npm run build:package
the autoprefixing seems to be different.I'll need to do some cross browser testing to see the impact.
Update: I've tested cross browser, and can't see a difference...
https://trello.com/c/Sg2Uiq9z/1583-update-frontend-dependencies