Skip to content

Update: remove unused CI#764

Merged
vcaesar merged 1 commit intomasterfrom
bitmap-pr
Apr 5, 2026
Merged

Update: remove unused CI#764
vcaesar merged 1 commit intomasterfrom
bitmap-pr

Conversation

@vcaesar
Copy link
Copy Markdown
Member

@vcaesar vcaesar commented Apr 5, 2026

  • Issues: #

Provide test code:

Description

...

Summary by CodeRabbit

  • Chores
    • Removed automated continuous integration configurations.

Copilot AI review requested due to automatic review settings April 5, 2026 07:19
@gemini-code-assist
Copy link
Copy Markdown

Important

Installation incomplete: to start using Gemini Code Assist, please ask the organization owner(s) to visit the Gemini Code Assist Admin Console and sign the Terms of Services.

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 5, 2026

Caution

Review failed

Pull request was closed or merged during review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 20fb9f1b-fefc-4e91-8b61-85b80dccacdf

📥 Commits

Reviewing files that changed from the base of the PR and between ded61ca and f33ac13.

📒 Files selected for processing (2)
  • .travis.yml
  • appveyor.yml
💤 Files with no reviewable changes (2)
  • .travis.yml
  • appveyor.yml

📝 Walkthrough

Walkthrough

This pull request removes continuous integration configuration files for Travis CI and AppVeyor, eliminating the automated testing setup that previously ran tests across Linux, macOS, and Windows environments using different CI platforms.

Changes

Cohort / File(s) Summary
CI Configuration Removal
.travis.yml, appveyor.yml
Deleted Travis CI configuration (Go 1.19.x and tip matrix, Linux/macOS setup) and AppVeyor configuration (Windows Visual Studio 2017 build, Go toolchain management, test execution pipeline).

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Suggested labels

update

Poem

🐰 The CI pipelines fade away,
No more Travis tests each day,
AppVeyor's build scripts disappear,
Simplified workflows now are clear!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Description check ⚠️ Warning The description is largely incomplete. It contains only placeholder content from the template ('Issues: #', empty test code block, and '...') without explaining the rationale or changes. Add a proper description explaining why these CI configurations are being removed and what the impact is. Include relevant issue links if applicable.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately identifies the main change—removal of Travis CI and AppVeyor CI configurations—which is the primary purpose of this changeset.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bitmap-pr

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@vcaesar vcaesar merged commit 787ff6d into master Apr 5, 2026
5 of 9 checks passed
Copy link
Copy Markdown

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 removes legacy CI configuration files for AppVeyor and Travis, presumably because the project no longer uses those services and relies on other CI (e.g., GitHub Actions / CircleCI).

Changes:

  • Delete appveyor.yml (AppVeyor pipeline config)
  • Delete .travis.yml (Travis CI pipeline config)

Reviewed changes

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

File Description
appveyor.yml Removes AppVeyor CI configuration from the repo
.travis.yml Removes Travis CI configuration from the repo
Comments suppressed due to low confidence (2)

appveyor.yml:1

  • This PR removes the AppVeyor config, but the main README still displays an AppVeyor build status badge (README.md line 8). Please remove/update that badge so users don’t think AppVeyor is still used for CI.
    .travis.yml:1
  • This PR removes the Travis CI config, but docs/archive/README_zh.md still includes a Travis build status badge/link (line 7). Please remove/update that reference to avoid pointing readers to a CI provider that’s no longer used.

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

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