Skip to content

Replace local packages/linting with published torrust-linting crate#417

Merged
josecelano merged 1 commit intomainfrom
416-replace-local-linting-with-published-crate
Apr 6, 2026
Merged

Replace local packages/linting with published torrust-linting crate#417
josecelano merged 1 commit intomainfrom
416-replace-local-linting-with-published-crate

Conversation

@josecelano
Copy link
Copy Markdown
Member

Overview

Closes #416

The packages/linting workspace package has been extracted and published to
crates.io as torrust-linting (v0.1.0).
This PR removes the redundant local copy and switches to the published crate.

Changes

Cargo.toml

  • Removed "packages/linting" from [workspace] members
  • Changed torrust-linting = { path = "packages/linting" }torrust-linting = "0.1.0"

packages/linting/ (deleted)

  • Removed the entire local package directory (15 files)

Documentation updated

  • packages/README.md — removed local linting section; updated library crates example to use crates.io version
  • docs/codebase-architecture.md — updated workspace packages listing
  • docs/contributing/linting.md — updated linting framework reference
  • docs/contributing/testing/quality/coverage.md — updated linting package location
  • .github/skills/dev/git-workflow/run-linters/skill.md — updated link
  • .github/skills/dev/git-workflow/run-linters/references/linters.md — updated framework architecture section

Verification

  • cargo build — passes (downloads torrust-linting v0.1.0 from crates.io)
  • cargo test — 416 passed, 0 failed
  • All pre-commit checks pass (linters, E2E infrastructure, E2E deployment)
  • No source code changes required — the published API is identical to the local package

Specification

See docs/issues/416-replace-local-linting-with-published-crate.md

@josecelano josecelano self-assigned this Apr 6, 2026
@josecelano
Copy link
Copy Markdown
Member Author

ACK 91b3961

@josecelano josecelano merged commit b47f6aa into main Apr 6, 2026
35 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.

Replace local packages/linting with published torrust-linting crate

1 participant