microsoft / axe-windows

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

chore: Complete TSA migration #901

Closed DaveTryon closed 1 year ago

DaveTryon commented 1 year ago

Details

This should be the last PR in the compliance upgrade. We previously added the TSAV2* pipeline variables to enable migration from TSA V1 to TSA V2. Now that the migration is complete, we want to drop the V2 from these variables and maintain just one set of variables. The pipeline variables have already been updated.

The TSA changes were validated by comparing the tsa.config files that are generated across 3 signed builds. Links below lead to the tsa.config files in the pipeline (I didn't copy them here to protect the data that they contain). The ComplianceDebug tsa.config files are unique in all 3 builds. Similarly, the SignedRelease tsa.config files are unique in all 3 builds.

main (using old variables) main (using updated variables) this PR (using updated variables)
ComplianceDebug ComplianceDebug ComplianceDebug
SignedRelease SignedRelease SignedRelease
Motivation

Pipeline maintenance

Context

Pull request checklist