-
Notifications
You must be signed in to change notification settings - Fork 22
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
Prepare site for 2020 #385 #390
Conversation
This pull request is automatically deployed with Now. Latest deployment for this branch: https://jsheroesio-git-385preparefor2020.clujsers.now.sh |
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.
looks great! I'll take some time in the weekend to fill some data there. I'll merge it when I have that data.
describe('Website navigation', () => { | ||
it('is a11y compliant', () => { | ||
// TODO - Will be fixed later | ||
it.skip('is a11y compliant', () => { |
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.
anything in particular failing in this test? I could have a look in the weekend
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.
i don't know why it fails...
fixes #388 #386
#387 is not done in this commit
additional fixes:
added .editorconfig for keeping the formatting style
fixed jest & cypress warnings in vscode (it, cy globals)
fixed dependency issues.
commented
the tests for a11y - i didn't knew what was the problem there.