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

Change assertion priority to only "required" and "optional" #117

Closed
zcorpan opened this issue Mar 11, 2020 · 2 comments
Closed

Change assertion priority to only "required" and "optional" #117

zcorpan opened this issue Mar 11, 2020 · 2 comments
Labels
feedback Issue raised by or for collecting input from people outside the core project team. tests About assistive technology tests

Comments

@zcorpan
Copy link
Member

zcorpan commented Mar 11, 2020

From #111, feedback from Vispero was that it may be hard to get agreement on assertion priority between "should have" and "optional", and we should merge them into only "optional".

I have implemented this change in working mode doc: #41

Assertion priority: Specifies whether an assertion is “must have” or “optional”.

What was the rationale to have "should have" originally? Do people agree that dropping it is a good idea?

@zcorpan zcorpan added feedback Issue raised by or for collecting input from people outside the core project team. tests About assistive technology tests labels Mar 11, 2020
@zcorpan zcorpan added the Agenda+Community Group To discuss in the next workstream summary meeting (usually the last teleconference of the month) label Mar 11, 2020
@zcorpan
Copy link
Member Author

zcorpan commented Mar 25, 2020

@mcking65 suggests "required" and "optional"

@zcorpan zcorpan removed the Agenda+Community Group To discuss in the next workstream summary meeting (usually the last teleconference of the month) label Mar 26, 2020
@zcorpan zcorpan changed the title Change assertion priority to only "must have" and "optional" Change assertion priority to only "required" and "optional" Apr 1, 2020
@zcorpan
Copy link
Member Author

zcorpan commented May 12, 2020

This should be fixed now.

@zcorpan zcorpan closed this as completed May 12, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feedback Issue raised by or for collecting input from people outside the core project team. tests About assistive technology tests
Projects
None yet
Development

No branches or pull requests

1 participant