Skip to content

Conversation

@CanerKaraca23
Copy link
Contributor

@CanerKaraca23 CanerKaraca23 commented Jan 3, 2026

  • Updated action versions.
  • Removed unnecessary steps.

Copilot AI review requested due to automatic review settings January 3, 2026 20:45
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 updates GitHub Actions workflow versions and removes unnecessary build steps across three workflow files. The changes aim to modernize the CI/CD pipeline by upgrading action versions, removing the obsolete MSVC setup (since the project uses Clang), and eliminating the submodules parameter (as the project uses CMake FetchContent instead of git submodules).

  • Updates various GitHub Actions to newer versions (checkout, github-script, upload/download-artifact)
  • Removes MSVC environment setup steps from CI and nightly workflows
  • Removes submodules checkout parameter from nightly workflow
  • Adds descriptive names to checkout steps and package version checks

Reviewed changes

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

File Description
.github/workflows/pr-comment.yml Updates actions/github-script from v7 to v8
.github/workflows/nightly.yml Updates multiple actions (checkout, github-script, upload/download-artifact, gh-release); removes MSVC setup and submodules parameter; adds descriptive step names
.github/workflows/ci.yml Updates checkout and upload-artifact actions; removes MSVC setup and manual package installation; adds version check step

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

@xesdoog
Copy link
Contributor

xesdoog commented Jan 4, 2026

AI slop should be banned.

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