Skip to content

Update Sonic#20154

Merged
seanbudd merged 1 commit into
nvaccess:masterfrom
OzancanKaratas:UpdateSonic
May 18, 2026
Merged

Update Sonic#20154
seanbudd merged 1 commit into
nvaccess:masterfrom
OzancanKaratas:UpdateSonic

Conversation

@OzancanKaratas
Copy link
Copy Markdown
Collaborator

Link to issue number:

None. Maybe related to #20103

Summary of the issue:

Sonic has been updated to the latest commit.

Description of user facing changes:

None, this is a developer/backend changes.

Description of developer facing changes:

None

Description of development approach:

Only new files were fetched. Changes were made to the Sonic's build system. It is believed that NVDA's build system does not need to be updated.

Testing strategy:

Compile NVDA. The compilation must be successful.

Known issues with pull request:

None

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.

If the GitHub Actions tests and build are successful, the pull request can be merged.

@OzancanKaratas OzancanKaratas requested a review from a team as a code owner May 17, 2026 18:21
@OzancanKaratas OzancanKaratas requested a review from seanbudd May 17, 2026 18:21
@seanbudd seanbudd merged commit 2beb4e7 into nvaccess:master May 18, 2026
36 of 39 checks passed
@github-actions github-actions Bot added this to the 2026.3 milestone May 18, 2026
Boumtchack pushed a commit to France-Travail/nvda that referenced this pull request May 18, 2026
@OzancanKaratas OzancanKaratas deleted the UpdateSonic branch May 18, 2026 17:52
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.

2 participants