Skip to content

Migrate CI/CD from AppVeyor to GitHub Actions#49

Merged
sparkeh9 merged 9 commits intomasterfrom
copilot/migrate-to-github-actions
Dec 15, 2025
Merged

Migrate CI/CD from AppVeyor to GitHub Actions#49
sparkeh9 merged 9 commits intomasterfrom
copilot/migrate-to-github-actions

Conversation

Copy link
Contributor

Copilot AI commented Nov 22, 2025

Replaces AppVeyor build pipeline with GitHub Actions workflow.

Changes

  • Created .github/workflows/build.yml: Replicates AppVeyor build process (restore, build Release, pack NuGet, upload artifact)
  • Updated README.md: Replaced AppVeyor badge with GitHub Actions badge
  • Removed appveyor.yml: Deleted legacy configuration and solution file reference
  • Migrated to Linux: Changed runner from windows-latest to ubuntu-latest (no Windows dependencies found)

Workflow Configuration

  • Runs on ubuntu-latest with .NET SDK from global.json (8.0.416)
  • Triggers on push/PR to master
  • Minimal permissions (contents: read)
  • Uploads CoreFtp.*.nupkg as workflow artifact
jobs:
  build:
    runs-on: ubuntu-latest
    permissions:
      contents: read
    steps:
      - uses: actions/checkout@v4
      - uses: actions/setup-dotnet@v4
      - run: dotnet restore
      - run: dotnet build CoreFtp.sln --configuration Release --no-restore
      - run: dotnet pack src/CoreFtp/CoreFtp.csproj --no-build --configuration Release
      - uses: actions/upload-artifact@v4

Verification

  • ✅ Project has no Windows-specific dependencies
  • ✅ Both netstandard1.6 and net452 targets build successfully on Linux
  • ✅ Build and pack complete successfully on ubuntu-latest
Original prompt

Migrate from AppVeyor to running in Github build actions.


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Copilot AI self-assigned this Nov 22, 2025
Copilot AI and others added 4 commits November 22, 2025 01:27
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Copilot AI changed the title [WIP] Migrate build process from AppVeyor to GitHub Actions Migrate CI/CD from AppVeyor to GitHub Actions Nov 22, 2025
Copilot AI requested a review from sparkeh9 November 22, 2025 01:33
@sparkeh9 sparkeh9 marked this pull request as ready for review November 24, 2025 14:04
Copilot AI review requested due to automatic review settings November 24, 2025 14:04
Copy link
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 successfully migrates the CI/CD pipeline from AppVeyor to GitHub Actions, modernizing the build infrastructure while maintaining the same build process. The migration also fixes a path casing issue that existed in the legacy AppVeyor configuration.

Key changes:

  • Replaced AppVeyor with GitHub Actions workflow using modern action versions (v4)
  • Corrected project path from incorrect CoreFTP to actual CoreFtp casing
  • Updated README badge to point to new GitHub Actions build status

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 1 comment.

File Description
.github/workflows/build.yml New GitHub Actions workflow replicating AppVeyor build steps (restore, build, pack, upload artifact) with .NET 8.0
appveyor.yml Removed legacy AppVeyor configuration file
README.md Updated build status badge from AppVeyor to GitHub Actions
CoreFtp.sln Removed reference to deleted appveyor.yml file from solution items

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@sparkeh9
Copy link
Owner

@copilot There's no reason to run the builds here on windows any more. We can just build this on Linux now - check there are no hard windows dependencies, then migrate the codebase to use a default Linux distro

Copilot AI and others added 3 commits December 15, 2025 10:22
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Co-authored-by: sparkeh9 <5622386+sparkeh9@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Dec 15, 2025

@copilot There's no reason to run the builds here on windows any more. We can just build this on Linux now - check there are no hard windows dependencies, then migrate the codebase to use a default Li...

Done! Verified that the project has no Windows dependencies and migrated the workflow to use ubuntu-latest. The build completes successfully on Linux, including both the netstandard1.6 and net452 targets. Changes committed in bb2ce1d.

@sparkeh9 sparkeh9 merged commit 2b76bae into master Dec 15, 2025
1 check passed
@sparkeh9 sparkeh9 deleted the copilot/migrate-to-github-actions branch December 15, 2025 10:41
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