Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Add GitHub Actions CI to replace AppVeyor and fix build errors #44

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Copilot wants to merge 4 commits into main
base: main
Choose a base branch
Loading
from copilot/add-github-ci-solution-2

Conversation

Copy link
Contributor

Copilot AI commented Oct 11, 2025
edited
Loading

Overview

This PR adds a modern GitHub Actions CI workflow to replace the existing AppVeyor CI configuration, while also fixing critical build errors that prevented the project from compiling.

Changes

1. Fixed Build Error in ExcelConstants.cs

The project had a syntax error in the CommonDateTimeFormats array initialization that prevented compilation:

// Before (incorrect)
public static readonly string[] CommonDateTimeFormats = 
 "yyyy-MM-ddTHH:mm:ss",
 "yyyy-MM-ddTHH:mm:ssZ",
 ...
// After (correct)
public static readonly string[] CommonDateTimeFormats = 
[
 "yyyy-MM-ddTHH:mm:ss",
 "yyyy-MM-ddTHH:mm:ssZ",
 ...
];

2. Updated GitHub Actions Workflow

Replaced the outdated .github/workflows/dotnet-core.yml (which used .NET Core 3.1) with a comprehensive modern CI pipeline:

Key improvements:

  • Multi-framework support: Builds and tests with .NET 6.0, 8.0, and 9.0
  • Cross-platform testing: Matrix strategy runs tests on both Ubuntu and Windows
  • Test result artifacts: Automatically uploads test results for review
  • NuGet package generation: Creates and uploads NuGet packages as artifacts
  • Updated actions: Uses latest versions (checkout@v4, setup-dotnet@v4, upload-artifact@v4)
  • Branch flexibility: Triggers on both main and master branches

3. Updated Documentation

  • Updated CI badge in README.md (Chinese) to reference GitHub Actions instead of AppVeyor
  • Updated CI badge in README_EN.md (English) to reference GitHub Actions instead of AppVeyor

4. Removed AppVeyor Configuration

  • Deleted appveyor.yml as it's replaced by GitHub Actions
  • Updated solution file to reference the new GitHub Actions workflow

Test Results

All 88 tests pass successfully
Builds successfully for all target frameworks: netstandard2.0, net472, net6.0, netstandard2.1, net8.0, net9.0
NuGet package generation verified

Migration Notes

The new GitHub Actions workflow provides better integration with the GitHub ecosystem and more comprehensive CI coverage than the previous AppVeyor setup. No action is required from contributors - the new CI will automatically run on all pull requests and pushes.

Original prompt

添加 github ci 方案 取代现有ci 并保证项目编译测试通过


💬 Share your feedback on Copilot coding agent for the chance to win a 200ドル gift card! Click here to start the survey.

Copilot AI and others added 3 commits October 11, 2025 09:11
Co-authored-by: chsword <274085+chsword@users.noreply.github.com>
Co-authored-by: chsword <274085+chsword@users.noreply.github.com>
...ons workflow
Co-authored-by: chsword <274085+chsword@users.noreply.github.com>
Copilot AI changed the title (削除) [WIP] Add GitHub CI solution to replace existing CI (削除ここまで) (追記) Add GitHub Actions CI to replace AppVeyor and fix build errors (追記ここまで) Oct 11, 2025
Copilot AI requested a review from chsword October 11, 2025 09:16
@chsword chsword marked this pull request as ready for review October 11, 2025 12:05
Copilot AI review requested due to automatic review settings October 11, 2025 12:05
Copy link

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 modernizes the CI/CD pipeline by replacing AppVeyor with GitHub Actions while fixing a critical build error that prevented compilation.

  • Replaced outdated AppVeyor CI with a comprehensive GitHub Actions workflow supporting multiple .NET versions (6.0, 8.0, 9.0) and cross-platform testing
  • Fixed syntax error in ExcelConstants.cs array initialization that prevented project compilation
  • Updated CI badges in README files to reference the new GitHub Actions workflow

Reviewed Changes

Copilot reviewed 6 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
appveyor.yml Removed outdated AppVeyor configuration
.github/workflows/dotnet-core.yml Added modern GitHub Actions CI workflow with multi-framework and cross-platform support
Chsword.Excel2Object/Internal/ExcelConstants.cs Fixed array initialization syntax error preventing compilation
README.md Updated CI badge to reference GitHub Actions
README_EN.md Updated CI badge to reference GitHub Actions
Chsword.Excel2Object.sln Updated solution items to reference new GitHub Actions workflow

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

+1 more reviewer

@chsword chsword chsword approved these changes

Reviewers whose approvals may not affect merge requirements

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

AltStyle によって変換されたページ (->オリジナル) /