[PR #2229] [MERGED] Fixed checking cppcheck version in ci.yml #2399

New issue

Closed

opened 2026-03-04 02:05:20 +03:00 by kerem · 0 comments

kerem commented

2026-03-04 02:05:20 +03:00

Owner

📋 Pull Request Information

Original PR: https://github.com/s3fs-fuse/s3fs-fuse/pull/2229
Author: @ggtakec
Created: 7/23/2023
Status: ✅ Merged
Merged: 7/24/2023
Merged by: @gaul

Base: master ← Head: fix_cppcheck_version

📝 Commits (1)

f0ad352 Fixed checking cppcheck version in ci.yml

📊 Changes

1 file changed (+6 additions, -3 deletions)

View changed files

📝 .github/workflows/ci.yml (+6 -3)

📄 Description

Relevant Issue (if applicable)

n/a

Details

Tighter version number checking in cppcheck.
The current decision statement bypasses the check for cases like "2.9".
This statement provides some protection against future version number format changes.(cppcheck on macOS is 3 digits)

And the same conditions were set for the CI part when supporting macOS fuse-t.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/s3fs-fuse/s3fs-fuse/pull/2229 **Author:** [@ggtakec](https://github.com/ggtakec) **Created:** 7/23/2023 **Status:** ✅ Merged **Merged:** 7/24/2023 **Merged by:** [@gaul](https://github.com/gaul) **Base:** `master` ← **Head:** `fix_cppcheck_version` --- ### 📝 Commits (1) - [`f0ad352`](https://github.com/s3fs-fuse/s3fs-fuse/commit/f0ad352d0e45c5543e419f8f38e2d8a8c1f4c7d3) Fixed checking cppcheck version in ci.yml ### 📊 Changes **1 file changed** (+6 additions, -3 deletions) <details> <summary>View changed files</summary> 📝 `.github/workflows/ci.yml` (+6 -3) </details> ### 📄 Description ### Relevant Issue (if applicable) n/a ### Details Tighter version number checking in cppcheck. The current decision statement bypasses the check for cases like "2.9". This statement provides some protection against future version number format changes.(cppcheck on macOS is 3 digits) And the same conditions were set for the CI part when supporting macOS fuse-t. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>