microsoft / axe-windows

Automated accessibility testing engine for Windows applications
MIT License
136 stars 62 forks source link

chore: Remove unnecessary error codes #761

Closed sfoslund closed 2 years ago

sfoslund commented 2 years ago

Details

Follow up on PR comment: https://github.com/microsoft/axe-windows/pull/759#issuecomment-1275385362, removing error codes that are no longer relevant

Motivation

Clean up code

Pull request checklist

codecov-commenter commented 2 years ago

Codecov Report

Merging #761 (da4c801) into main (4fc19c3) will increase coverage by 0.87%. The diff coverage is 79.15%.

:exclamation: Current head da4c801 differs from pull request most recent head 6faa959. Consider uploading reports for the commit 6faa959 to get more accurate results

@@            Coverage Diff             @@
##             main     #761      +/-   ##
==========================================
+ Coverage   73.70%   74.57%   +0.87%     
==========================================
  Files         398      420      +22     
  Lines       12046    12895     +849     
==========================================
+ Hits         8878     9616     +738     
- Misses       3168     3279     +111     
Impacted Files Coverage Δ
src/Actions/Actions/ControlPatternAction.cs 0.00% <0.00%> (ø)
src/Actions/Actions/CustomUIAAction.cs 0.00% <0.00%> (ø)
src/Actions/Actions/ListenAction.cs 0.00% <0.00%> (ø)
src/Actions/Actions/LoadActionParts.cs 90.90% <ø> (ø)
src/Actions/Actions/SetDataAction.cs 0.00% <0.00%> (ø)
...rc/Actions/Attributes/InteractionLevelAttribute.cs 0.00% <ø> (ø)
src/Actions/Contexts/ElementDataContext.cs 86.36% <ø> (ø)
src/Actions/Misc/ExtensionMethods.cs 7.46% <0.00%> (-31.60%) :arrow_down:
src/Actions/Resources/ErrorMessages.Designer.cs 0.00% <ø> (-55.56%) :arrow_down:
src/Actions/Trackers/BaseTracker.cs 51.02% <0.00%> (-16.55%) :arrow_down:
... and 230 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

sfoslund commented 2 years ago

Amending last commit to retrigger checks, as license check was stuck. No real changes.