Skip to content

Disable AppVeyor for PRs/snapshots#18555

Merged
seanbudd merged 1 commit intomasterfrom
disableAppVeyor
Jul 29, 2025
Merged

Disable AppVeyor for PRs/snapshots#18555
seanbudd merged 1 commit intomasterfrom
disableAppVeyor

Conversation

@seanbudd
Copy link
Copy Markdown
Member

@seanbudd seanbudd commented Jul 25, 2025

Link to issue number:

Part of #17878

Summary of the issue:

As we transition from AppVeyor to GitHub Actions, we are nearly ready to deploy snapshots and releases via GHA.
For safety, we will perform tagged releases in the 2025.2 release cycle with AppVeyor still.

Description of user facing changes:

none

Description of developer facing changes:

AppVeyor will only build tagged releases.
PR builds, try builds and snapshots will now be built by GitHub actions.

Description of development approach:

removed lines from appveyor.yml. Appveyor uses master's copy of this file for all releases, unlike GitHub actions where it depends on the branches involved.

Testing strategy:

Not needed

Known issues with pull request:

blocked currently

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

@seanbudd seanbudd added the conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review. label Jul 29, 2025
@seanbudd seanbudd marked this pull request as ready for review July 29, 2025 05:31
Copilot AI review requested due to automatic review settings July 29, 2025 05:31
@seanbudd seanbudd requested a review from a team as a code owner July 29, 2025 05:31
@seanbudd seanbudd requested a review from SaschaCowley July 29, 2025 05:31
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 pull request removes branches from the AppVeyor CI configuration to transition from AppVeyor to GitHub Actions for building NVDA. The change restricts AppVeyor to only build tagged releases while PR builds, try builds, and snapshots will be handled by GitHub Actions.

  • Removes master, beta, rc, and try-* branches from AppVeyor's build trigger list
  • Retains only release-* branches for AppVeyor builds

@seanbudd
Copy link
Copy Markdown
Member Author

testing has been performed, this is ready to merge tomorrow with server changes

@seanbudd seanbudd added this to the 2025.3 milestone Jul 29, 2025
@seanbudd seanbudd merged commit 239052f into master Jul 29, 2025
18 of 19 checks passed
@seanbudd seanbudd deleted the disableAppVeyor branch July 29, 2025 23:25
seanbudd added a commit that referenced this pull request Aug 6, 2025
Takes #18555 to beta to prevent beta/rc PRs and snapshots from building on appveyor
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

conceptApproved Similar 'triaged' for issues, PR accepted in theory, implementation needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants