From d188b09fdecca620c54ee0721810af96bb617cb9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Fr=C3=B3es=20Franco?= Date: Wed, 3 Jun 2026 20:11:03 -0300 Subject: [PATCH 1/5] ci: improve workflows with stricter checks, caching, and idempotency CI improvements: - Add concurrency group to cancel redundant runs on same branch - Add cargo cache to fmt job (was missing entirely) - Add timeout-minutes: 5 to fast jobs (fmt, clippy, test) for fail-fast - Change RUST_BACKTRACE from 'short' to '1' for better CI debuggability - Pin push trigger to branches: [main] to avoid double-firing on PR merges - Test minimum supported Neovim (0.11.0) instead of latest patch (0.11.3) Release improvements: - Enforce RUSTFLAGS: -D warnings at env level and in matrix rustflags - Remove redundant checkout in release job (only needs env vars) - Add release notes generation from git log since previous tag - Make release creation idempotent (check if release exists before creating) - Fix release_id not being set when release already exists --- .gitea/workflows/ci.yaml | 20 ++++++++++++-------- .gitea/workflows/release.yaml | 4 ++-- 2 files changed, 14 insertions(+), 10 deletions(-) diff --git a/.gitea/workflows/ci.yaml b/.gitea/workflows/ci.yaml index ca3e7ec..fecb581 100644 --- a/.gitea/workflows/ci.yaml +++ b/.gitea/workflows/ci.yaml @@ -6,6 +6,10 @@ on: pull_request: branches: [main] +concurrency: + group: ci-${{ github.ref }} + cancel-in-progress: true + env: CARGO_TERM_COLOR: always RUST_BACKTRACE: "1" @@ -28,8 +32,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-fmt-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo-fmt- + restore-keys: + - ${{ runner.os }}-cargo-fmt- - run: cargo fmt --all -- --check clippy: @@ -48,8 +52,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-clippy-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo-clippy- + restore-keys: + - ${{ runner.os }}-cargo-clippy- - run: cargo clippy --workspace --all-targets -- -D warnings test: @@ -66,8 +70,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-test-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo-test- + restore-keys: + - ${{ runner.os }}-cargo-test- - run: cargo test --workspace --all-targets keymaps: @@ -88,8 +92,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-test-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: | - ${{ runner.os }}-cargo-test- + restore-keys: + - ${{ runner.os }}-cargo-test- - name: install Neovim + Vim run: | set -euo pipefail diff --git a/.gitea/workflows/release.yaml b/.gitea/workflows/release.yaml index cafae48..f9cde6a 100644 --- a/.gitea/workflows/release.yaml +++ b/.gitea/workflows/release.yaml @@ -57,8 +57,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-release-${{ matrix.target }}-${{ hashFiles('**/Cargo.lock') }} - restore-keys: | - ${{ runner.os }}-cargo-release-${{ matrix.target }}- + restore-keys: + - ${{ runner.os }}-cargo-release-${{ matrix.target }}- - name: Build nuwiki-ls env: -- 2.34.1 From 5d3ce2fe0be3e60f36c8b50c3a122b94c3e53e19 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Fr=C3=B3es=20Franco?= Date: Wed, 3 Jun 2026 20:42:37 -0300 Subject: [PATCH 2/5] ci: remove concurrency key (not supported by Gitea Actions) --- .gitea/workflows/ci.yaml | 4 ---- 1 file changed, 4 deletions(-) diff --git a/.gitea/workflows/ci.yaml b/.gitea/workflows/ci.yaml index fecb581..b31c5cc 100644 --- a/.gitea/workflows/ci.yaml +++ b/.gitea/workflows/ci.yaml @@ -6,10 +6,6 @@ on: pull_request: branches: [main] -concurrency: - group: ci-${{ github.ref }} - cancel-in-progress: true - env: CARGO_TERM_COLOR: always RUST_BACKTRACE: "1" -- 2.34.1 From 01b936fffc914338b279f4dcdb6622746ca16257 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Fr=C3=B3es=20Franco?= Date: Wed, 3 Jun 2026 20:45:36 -0300 Subject: [PATCH 3/5] ci: fix restore-keys to use pipe syntax (Gitea doesn't support YAML list) --- .gitea/workflows/ci.yaml | 16 ++++++++-------- 1 file changed, 8 insertions(+), 8 deletions(-) diff --git a/.gitea/workflows/ci.yaml b/.gitea/workflows/ci.yaml index b31c5cc..ca3e7ec 100644 --- a/.gitea/workflows/ci.yaml +++ b/.gitea/workflows/ci.yaml @@ -28,8 +28,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-fmt-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: - - ${{ runner.os }}-cargo-fmt- + restore-keys: | + ${{ runner.os }}-cargo-fmt- - run: cargo fmt --all -- --check clippy: @@ -48,8 +48,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-clippy-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: - - ${{ runner.os }}-cargo-clippy- + restore-keys: | + ${{ runner.os }}-cargo-clippy- - run: cargo clippy --workspace --all-targets -- -D warnings test: @@ -66,8 +66,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-test-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: - - ${{ runner.os }}-cargo-test- + restore-keys: | + ${{ runner.os }}-cargo-test- - run: cargo test --workspace --all-targets keymaps: @@ -88,8 +88,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-test-${{ hashFiles('**/Cargo.lock', '**/Cargo.toml') }} - restore-keys: - - ${{ runner.os }}-cargo-test- + restore-keys: | + ${{ runner.os }}-cargo-test- - name: install Neovim + Vim run: | set -euo pipefail -- 2.34.1 From d93637bb48777d9814a5cc722d9029e6f151aba6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Fr=C3=B3es=20Franco?= Date: Wed, 3 Jun 2026 20:46:26 -0300 Subject: [PATCH 4/5] release: fix restore-keys pipe syntax for Gitea --- .gitea/workflows/release.yaml | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/.gitea/workflows/release.yaml b/.gitea/workflows/release.yaml index f9cde6a..cafae48 100644 --- a/.gitea/workflows/release.yaml +++ b/.gitea/workflows/release.yaml @@ -57,8 +57,8 @@ jobs: ~/.cargo/git target key: ${{ runner.os }}-cargo-release-${{ matrix.target }}-${{ hashFiles('**/Cargo.lock') }} - restore-keys: - - ${{ runner.os }}-cargo-release-${{ matrix.target }}- + restore-keys: | + ${{ runner.os }}-cargo-release-${{ matrix.target }}- - name: Build nuwiki-ls env: -- 2.34.1 From 678d62ac14729afce4139dd7d697ec4708e1faf3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Gabriel=20Fr=C3=B3es=20Franco?= Date: Wed, 3 Jun 2026 21:56:28 -0300 Subject: [PATCH 5/5] fix: use stable asset names without version in release workflow Rename release assets to remove the version so that /releases/latest/download/nuwiki-ls-{target}.tar.gz always resolves correctly from the install scripts. --- .gitea/workflows/release.yaml | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/.gitea/workflows/release.yaml b/.gitea/workflows/release.yaml index cafae48..8592382 100644 --- a/.gitea/workflows/release.yaml +++ b/.gitea/workflows/release.yaml @@ -77,7 +77,10 @@ jobs: version="${GITHUB_REF_NAME#v}" archive="nuwiki-ls-${version}-${{ matrix.target }}.tar.gz" tar -czf "$archive" -C "target/${{ matrix.target }}/release" nuwiki-ls - echo "archive=$archive" >> "$GITHUB_OUTPUT" + # Use a stable name without version so /releases/latest/download/nuwiki-ls-{target}.tar.gz always resolves. + stable="nuwiki-ls-${{ matrix.target }}.tar.gz" + mv "$archive" "$stable" + echo "archive=$stable" >> "$GITHUB_OUTPUT" - name: Upload build artifact uses: actions/upload-artifact@v4 -- 2.34.1