-
Notifications
You must be signed in to change notification settings - Fork 544
Create accessability.md file #4616
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
Conversation
fixed stand in word
Updating the branch because one of our tests was failing and had to be disabled. I'm still seeing some weird behaviour on longtests but I don't think that will affect this PR since they shouldn't all run without code changes. |
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.
Okay, now that our tests aren't freaking out any more, looks like we've got a few spelling issues to fix before we can merge this.
It's complaining about the following:
inclusivity
GAAD
WCAG
You'll need to add them here to the spelling allow list and then the spell checker will stop complaining:
https://github.com/intel/cve-bin-tool/blob/main/.github/actions/spelling/allow.txt
Let me know if you need any help!
Hi @WhataTiberius are you still working on this one, should be a quick fix |
adding the words: inclusivity GAAD WCAG so that PR intel#4616 can merge
Updating the branch because #4726 should have resolved the spelling issues. (Thanks @vedpawar2254 !) |
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.
Glad we had a bit of time to chat about this. I'm going to go ahead and merge and we'll work on next steps. If you've got any immediate changes we need, remember we've got hackathon folk working on cve-bin-tool this week!
creating a accessibility file to describe CVE bin tool's commitment to GAAD pledge and accessibility