Skip to content

Azure Pipeline でのみ実装されている cppcheck を GitHub Actions に移植する#2084

Merged
m-tmatma merged 1 commit into
sakura-editor:masterfrom
m-tmatma:feature/actions-cppcheck
Jun 23, 2025
Merged

Azure Pipeline でのみ実装されている cppcheck を GitHub Actions に移植する#2084
m-tmatma merged 1 commit into
sakura-editor:masterfrom
m-tmatma:feature/actions-cppcheck

Conversation

@m-tmatma

Copy link
Copy Markdown
Member

ci/azure-pipelines/template.job.cppcheck.yml の JOB で実装されている処理を GitHub Actions に移植

PR対象

  • ビルド手順/CI

カテゴリ

  • 改善 (Azure PipeLine からの移植)

PR の背景

利用するCI/CDサービスをGitHub Actionsに一本化するために Azure Pipeline でのみ実装されている job を
GitHub Actions で実装しなおす。

仕様・動作説明

PR の影響範囲

GitHub Actions

テスト内容

CI で実行されている処理で cppcheck の artifacts が生成されること

関連 issue, PR

#1809

参考資料

cursor.sh を使って azure pipeline の job の移植の大部分を行いました。

@m-tmatma m-tmatma requested a review from Copilot June 22, 2025 03:59
@m-tmatma m-tmatma self-assigned this Jun 22, 2025
@m-tmatma m-tmatma added CI appveyor など CI 関連 【ChangeLog除外】 azure pipelines labels Jun 22, 2025

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

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 migrates the cppcheck job from Azure Pipeline to GitHub Actions to align CI/CD services on GitHub.

  • Introduces a new GitHub Actions workflow for cppcheck running on Windows with matrix configurations.
  • Implements installation, execution, artifact zipping, and artifact upload steps for cppcheck.
Comments suppressed due to low confidence (2)

.github/workflows/cppcheck.yml:39

  • Consider verifying that the installer path resolves correctly across environments or add a step to check for the existence of the installer before running the installation.
        run: msiexec /i externals\cppcheck\cppcheck-1.84-x64-Setup.msi /quiet /qn /norestart /log cppcheck-install.log

.github/workflows/cppcheck.yml:54

  • Ensure that the glob pattern '**Log.zip' precisely targets the intended log file(s) to avoid inadvertently including unrelated files in the artifacts.
          path: '**Log.zip'

@m-tmatma m-tmatma added the GitHub Actions GitHub Actions関連 label Jun 22, 2025
@sonarqubecloud

Copy link
Copy Markdown

- '.github/*'
- '.gitignore'
- '.editorconfig'
- 'appveyor.yml'

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

対応不要。

pathsで拡張子(cpp、h、hpp)を指定したほうが直接的で分かりやすいかも。

@m-tmatma m-tmatma merged commit b8981ec into sakura-editor:master Jun 23, 2025
10 checks passed
@m-tmatma m-tmatma deleted the feature/actions-cppcheck branch July 2, 2025 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

azure pipelines CI appveyor など CI 関連 【ChangeLog除外】 GitHub Actions GitHub Actions関連

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants