tests: replace workspace guards with repository linters #294

Merged

natsukium merged 1 commit from tooling/workspace-guards into main 2026-09-18 16:08:51 +09:00

Conversation 0 Commits 1 Files changed 15 +247 -674

natsukium commented 2026-09-18 15:18:12 +09:00

Owner

Copy link

Summary

• remove workspace tests for the optional documentation index and duplicated CLI exit-code list
• use the built-in git-hooks.nix lychee hook for Markdown links
• move evaluator bans to cargo-deny and retain the custom Rust source-reference guard

Verification

• cargo nextest run -p felis-workspace-tests --all-features --all-targets
• cargo clippy -p felis-cli -p felis-workspace-tests --all-targets --all-features -- -D warnings
• cargo deny check
• nix develop --command just docs-links
• nix develop --command treefmt --fail-on-change
• nix develop --command just prose-check

## Summary - remove workspace tests for the optional documentation index and duplicated CLI exit-code list - use the built-in git-hooks.nix lychee hook for Markdown links - move evaluator bans to cargo-deny and retain the custom Rust source-reference guard ## Verification - `cargo nextest run -p felis-workspace-tests --all-features --all-targets` - `cargo clippy -p felis-cli -p felis-workspace-tests --all-targets --all-features -- -D warnings` - `cargo deny check` - `nix develop --command just docs-links` - `nix develop --command treefmt --fail-on-change` - `nix develop --command just prose-check`

natsukium added 1 commit 2026-09-18 15:18:12 +09:00

tests: use repository linters for workspace guards ... 977154197b

Keep workspace tests for source citations while delegating Markdown link and dependency checks to lychee and cargo-deny. Drop the optional documentation index and duplicated CLI exit-code synchronization so the workspace test package only enforces invariants without an existing repository tool.

Assisted-by: Pi Coding Agent:GPT 5.6 Luna

natsukium scheduled this pull request to auto merge when all checks succeed 2026-09-18 15:20:41 +09:00

natsukium force-pushed tooling/workspace-guards from 977154197b to 1ad842bffd 2026-09-18 15:51:40 +09:00 Compare

natsukium merged commit 513de088f7 into main 2026-09-18 16:08:51 +09:00

natsukium deleted branch tooling/workspace-guards 2026-09-18 16:08:51 +09:00

natsukium referenced this pull request from a commit 2026-09-18 16:08:53 +09:00

tests: replace workspace guards with repository linters (#294)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

priority/P0

Must be resolved before the v0.1.0 tag

  

priority/P1

Must be settled before the first public contract freezes

  

priority/P2

Explicitly deferred until after v0.1.0

  

release/v0.1.0

Work tracked for the first tagged release

  

status/blocked

Cannot start now: waits on an upstream release, another issue, or a recorded revisit trigger

  

status/planned

Triaged with a source-grounded plan and a passed pi review; ready to implement

  

type/bug

Shipped behavior that contradicts the docs or the code's own intent; needs a product code change

  

type/design

An open scope or trade-off question; needs a recorded decision in the owning explanation doc before code

  

type/test-gap

Documented or reviewed behavior that no test pins; usually split from a review finding, needs no product code change

  

type/tracker

Parent issue that tracks child work

No labels

priority/P0

priority/P1

priority/P2

release/v0.1.0

status/blocked

status/planned

type/bug

type/design

type/test-gap

type/tracker

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

1 participant

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

natsukium/felis!294

No description provided.