Skip to content

Test NVDA on server 2022#18755

Merged
seanbudd merged 7 commits intonvaccess:masterfrom
LeonarddeR:additionalRunners
Aug 19, 2025
Merged

Test NVDA on server 2022#18755
seanbudd merged 7 commits intonvaccess:masterfrom
LeonarddeR:additionalRunners

Conversation

@LeonarddeR
Copy link
Copy Markdown
Collaborator

@LeonarddeR LeonarddeR commented Aug 19, 2025

Link to issue number:

Requested in #18708 (comment)
Related to #18684

Summary of the issue:

it was requested to run system tests on Windows Server 2022.

Description of user facing changes:

None

Description of developer facing changes:

NVDA is now systemd tested on Server 2022.

Description of development approach:

  • Added additional runner to the GitHub Actions matrix.
  • Excluded symbols on windows-2022 with an explanatory comment.
  • Visual Studio code intellisence whined about actions/checkout@v4, updated to v5

Testing strategy:

System tests

Known issues with pull request:

Nothing new apart from the symbols exclusion.

Code Review Checklist:

  • Documentation:
    • Change log entry
    • User Documentation
    • Developer / Technical Documentation
    • Context sensitive help for GUI changes
  • Testing:
    • Unit tests
    • System (end to end) tests
    • Manual testing
  • UX of all users considered:
    • Speech
    • Braille
    • Low Vision
    • Different web browsers
    • Localization in other languages / culture than English
  • API is compatible with existing add-ons.
  • Security precautions taken.

@coderabbitai summary

@LeonarddeR
Copy link
Copy Markdown
Collaborator Author

I don't understand why this fails. It looks like there's an oddity with the cache.

@LeonarddeR
Copy link
Copy Markdown
Collaborator Author

Never mind, let's remove 2019. it is no longer supported. See actions/runner-images#12045

@LeonarddeR LeonarddeR changed the title Test NVDA on server 2019 and 2022 Test NVDA on server 2022 Aug 19, 2025
@LeonarddeR LeonarddeR marked this pull request as ready for review August 19, 2025 17:59
Copilot AI review requested due to automatic review settings August 19, 2025 17:59
@LeonarddeR LeonarddeR requested a review from a team as a code owner August 19, 2025 17:59
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds Windows Server 2022 as a supported runner for GitHub Actions workflows to expand system test coverage. The change enables NVDA testing on an additional Windows environment while addressing compatibility issues.

Key changes:

  • Adds Windows Server 2022 to the supported runners matrix
  • Updates GitHub Actions checkout action to v5
  • Excludes symbol pronunciation tests on Windows 2022 due to a known Notepad focus issue

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@seanbudd
Copy link
Copy Markdown
Member

Yeah we were having issues with the cache yesterday - I think related to hitting storage limits. I think they are resolved now?

Copy link
Copy Markdown
Member

@seanbudd seanbudd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, and neat fix with matrix.exclude

@seanbudd seanbudd enabled auto-merge (squash) August 19, 2025 23:22
@seanbudd seanbudd merged commit 9f23ecf into nvaccess:master Aug 19, 2025
44 of 47 checks passed
@github-actions github-actions bot added this to the 2026.1 milestone Aug 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants