Skip to content

ci: add CI and version suggestion workflows - #6

Merged
debba merged 2 commits into
mainfrom
ci/add-ci-workflows
Oct 12, 2026
Merged

debba merged 2 commits into
mainfrom
ci/add-ci-workflows

Conversation

@debba

@debba debba commented Oct 11, 2026 •

Copy link
Copy Markdown
Collaborator

Brings this repo in line with the postgresql and sqlserver plugins, which until now were the only ones with CI beyond the release workflow.

Workflows

  • ci.yml
    • test: cargo build/cargo test --locked, clippy -D warnings, cargo fmt --check
    • validate-manifest: checks .tabularium against the live registry schema
    • pr-title: Conventional Commits check
    • markdownlint
    • audit: rustsec/audit-check on every push/PR plus a weekly scheduled run
  • version-suggestion.yml: same as postgresql, comments the suggested next tag on each PR. The prerelease:* label is optional here: without it the suggestion targets a stable release, so PRs don't fail for a missing label.

Other changes

  • src/main.rs: cargo fmt plus cargo clippy --fix (19 needless_borrow). No behaviour change, tests still pass.
  • .markdownlint.json / .markdownlintignore copied from the other plugins. CHANGELOG.md is ignored because git-cliff regenerates it on every release.
  • Cargo.lock: semver-compatible bumps of rustls (0.23.45), rustls-webpki (0.103.15), aws-lc-sys (0.45.0) and lz4_flex (0.11.6, the previous one was also yanked). The first audit run flagged 8 advisories on the old versions (RUSTSEC-2026-0041, -0044, -0048, -0049, -0098, -0099, -0104, -0285).
  • README: alt text on images, blank line after the title, table indented under its list item.

Verified locally: tests, clippy, fmt, manifest validation and markdownlint all pass.

Port the CI setup used by the postgresql and sqlserver plugins: tests, clippy, rustfmt, .tabularium validation, Conventional Commits PR titles, markdownlint and a weekly cargo audit. Apply cargo fmt and clippy fixes so the new checks start green.
@kilo-code-bot

kilo-code-bot Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Files Reviewed (7 files)
  • .github/workflows/ci.yml - New CI workflow (test, validate-manifest, pr-title, markdownlint, audit)
  • .github/workflows/version-suggestion.yml - Version suggestion via pull_request_target (safe, reads only event payload)
  • .markdownlint.json - Markdown lint config (MD013/MD033/MD041/MD060 disabled, MD024 siblings_only)
  • .markdownlintignore - Ignores target/, node_modules/, CHANGELOG.md
  • Cargo.lock - Security dependency updates (rustls, aws-lc-rs, aws-lc-sys, lz4_flex, rustls-webpki)
  • README.md - Added alt text to images/badges, fixed table indentation
  • src/main.rs - cargo fmt + clippy --fix (removed needless borrows, no behaviour change)
Previous Review Summary (commit 68a7b60)

Current summary above is authoritative. Previous snapshots are kept for context only.

Previous review (commit 68a7b60)

Status: No Issues Found | Recommendation: Merge

Files Reviewed (6 files)
  • .github/workflows/ci.yml - CI jobs: test, validate-manifest, pr-title, markdownlint, audit
  • .github/workflows/version-suggestion.yml - version-suggestion via pull_request_target
  • .markdownlint.json / .markdownlintignore - lint config, CHANGELOG ignored
  • README.md - alt text, blank line after title, table indentation
  • src/main.rs - cargo fmt + clippy --fix (needless_borrow), no behaviour change

Reviewed by free · Input: 176.4K · Output: 3K · Cached: 220.3K

Resolves RUSTSEC-2026-0041, -0044, -0048, -0049, -0098, -0099, -0104 and -0285 reported by the new audit job.
@debba
debba merged commit 4328a86 into main Oct 12, 2026
11 checks passed
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.

1 participant