Skip to content

ci: docs-only fast path (skip heavy jobs) #30

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Sep 24, 2025
Merged

ci: docs-only fast path (skip heavy jobs) #30

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
54 changes: 54 additions & 0 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -26,11 +26,44 @@ concurrency:
cancel-in-progress: true

jobs:
changes:
name: Detect Changes
runs-on: ubuntu-latest
outputs:
docs_only: ${{ steps.out.outputs.docs_only }}
steps:
- uses: actions/checkout@v4
- name: Paths filter
id: filter
uses: dorny/paths-filter@v3
with:
token: ${{ secrets.GITHUB_TOKEN }}
filters: |
docs:
- '**/*.md'
- 'docs/**'
- 'README.md'
- 'AGENTS.md'
- '.github/*.md'
Copy link

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

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

The pattern .github/*.md only matches markdown files directly in the .github directory, but not in subdirectories. Consider using .github/**/*.md to include markdown files in subdirectories like .github/workflows/ or other nested folders.

Suggested change
- '.github/*.md'
- '.github/**/*.md'

Copilot uses AI. Check for mistakes.

- 'PR_DESCRIPTIONS/**'
code:
Copy link

Copilot AI Sep 24, 2025

Choose a reason for hiding this comment

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

The code filter pattern !**/*.md will not properly detect code changes when markdown files are modified alongside code files. The paths-filter action requires at least one positive pattern for each filter. Consider adding explicit code patterns like - '**/*.rs', - '**/*.dart', etc., or use - '**' as the first pattern followed by exclusions.

Suggested change
code:
code:
- '**'

Copilot uses AI. Check for mistakes.

- '!**/*.md'
- name: Set outputs
id: out
run: |
if [ "${{ steps.filter.outputs.docs }}" = "true" ] && [ "${{ steps.filter.outputs.code }}" != "true" ]; then
echo "docs_only=true" >> "$GITHUB_OUTPUT"
else
echo "docs_only=false" >> "$GITHUB_OUTPUT"
fi
rustfmt-check:
name: Rustfmt Check
runs-on: ubuntu-latest
timeout-minutes: 10
continue-on-error: false
needs: [changes]
env:
DOCS_ONLY: ${{ needs.changes.outputs.docs_only }}

steps:
- uses: actions/checkout@v4
Expand All @@ -40,11 +73,13 @@ jobs:
toolchain: ${{ env.RUST_VERSION }}
components: rustfmt
- name: Check formatting (jive-api)
if: env.DOCS_ONLY != 'true'
working-directory: jive-api
run: |
cargo fmt --all -- --check

- name: Check formatting (jive-core)
if: env.DOCS_ONLY != 'true'
working-directory: jive-core
run: |
cargo fmt --all -- --check
Expand All @@ -54,15 +89,20 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 10
continue-on-error: false
needs: [changes]
env:
DOCS_ONLY: ${{ needs.changes.outputs.docs_only }}

steps:
- uses: actions/checkout@v4
- name: Install cargo-deny
if: env.DOCS_ONLY != 'true'
run: |
curl -sSfL https://github.com/EmbarkStudios/cargo-deny/releases/download/0.14.24/cargo-deny-0.14.24-x86_64-unknown-linux-musl.tar.gz | tar xz
sudo mv cargo-deny*/cargo-deny /usr/local/bin/cargo-deny
cargo-deny --version
- name: Run cargo-deny (API)
if: env.DOCS_ONLY != 'true'
working-directory: jive-api
run: |
set -o pipefail
Expand All @@ -79,17 +119,22 @@ jobs:
runs-on: ubuntu-latest
timeout-minutes: 20
continue-on-error: false
needs: [changes]
env:
DOCS_ONLY: ${{ needs.changes.outputs.docs_only }}

steps:
- uses: actions/checkout@v4

- name: Setup Flutter
if: env.DOCS_ONLY != 'true'
uses: subosito/flutter-action@v2
with:
flutter-version: ${{ env.FLUTTER_VERSION }}
channel: 'stable'

- name: Cache Flutter dependencies
if: env.DOCS_ONLY != 'true'
uses: actions/cache@v4
with:
path: |
Expand All @@ -101,15 +146,18 @@ jobs:
${{ runner.os }}-flutter-

- name: Install dependencies
if: env.DOCS_ONLY != 'true'
working-directory: jive-flutter
run: flutter pub get

- name: Generate code (build_runner)
if: env.DOCS_ONLY != 'true'
working-directory: jive-flutter
run: |
flutter pub run build_runner build --delete-conflicting-outputs || true

- name: Analyze code (non-fatal for now)
if: env.DOCS_ONLY != 'true'
working-directory: jive-flutter
run: |
set -o pipefail
Expand All @@ -124,6 +172,7 @@ jobs:
path: flutter-analyze-output.txt

- name: Run tests
if: env.DOCS_ONLY != 'true'
working-directory: jive-flutter
run: |
# Generate machine-readable test results (non-fatal for reporting)
Expand Down Expand Up @@ -178,6 +227,9 @@ jobs:
name: Rust API Tests
runs-on: ubuntu-latest
timeout-minutes: 20
needs: [changes]
env:
DOCS_ONLY: ${{ needs.changes.outputs.docs_only }}

services:
postgres:
Expand Down Expand Up @@ -208,12 +260,14 @@ jobs:
- uses: actions/checkout@v4

- name: Setup Rust
if: env.DOCS_ONLY != 'true'
uses: dtolnay/rust-toolchain@stable
with:
toolchain: ${{ env.RUST_VERSION }}
components: rustfmt, clippy

- name: Cache Rust dependencies
if: env.DOCS_ONLY != 'true'
uses: actions/cache@v4
with:
path: |
Expand Down