Skip to content

build: add ci workflow to verify c89 build and tests#119

Closed
tomghuang wants to merge 1 commit intomasterfrom
feature-add-c89-build-workflow
Closed

build: add ci workflow to verify c89 build and tests#119
tomghuang wants to merge 1 commit intomasterfrom
feature-add-c89-build-workflow

Conversation

@tomghuang
Copy link
Copy Markdown
Contributor

This patch introduces a workflow that ensures argtable3 can be built with GCC in strict C89 mode. The workflow builds both Debug and Release configurations and runs all unit tests for each. The workflow will fail if any build or test fails, guaranteeing C89 compatibility and test coverage on every change.

@tomghuang tomghuang requested a review from Copilot May 25, 2025 21:19
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

Adds a GitHub Actions workflow to enforce C89 compatibility by building and testing argtable3 in both Debug and Release modes.

  • Installs CMake, GCC, and Make on ubuntu-latest runners
  • Configures, builds (with -Wall -Wextra -pedantic) and runs tests in strict C89 for Debug and Release
  • Triggers on workflow_dispatch, pushes to master, and pull requests
Comments suppressed due to low confidence (1)

.github/workflows/c89.yml:7

  • [nitpick] Restrict pull_request to the master branch (e.g., pull_request: branches: [ master ]) to mirror the push trigger and avoid unintended runs on feature branches.
pull_request:

@tomghuang tomghuang force-pushed the feature-add-c89-build-workflow branch 2 times, most recently from c7d3f22 to 3cbf52d Compare May 25, 2025 21:40
This patch introduces a workflow that ensures argtable3 can be built
with GCC in strict C89 mode. The workflow builds both Debug and
Release configurations and runs all unit tests for each. The workflow
will fail if any build or test fails, guaranteeing C89 compatibility
and test coverage on every change.
@tomghuang tomghuang force-pushed the feature-add-c89-build-workflow branch from 3cbf52d to b243cff Compare May 25, 2025 21:43
@tomghuang
Copy link
Copy Markdown
Contributor Author

This pull request is not necessary because we already add -std=c89 in CMakeLists.txt.

@tomghuang tomghuang closed this May 25, 2025
@tomghuang tomghuang deleted the feature-add-c89-build-workflow branch May 25, 2025 21:55
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