Skip to content

ci: keep tests-only; remove coverage and security workflows#180

Merged
Slokh merged 4 commits intomainfrom
feature/ci-gates
Feb 26, 2026
Merged

ci: keep tests-only; remove coverage and security workflows#180
Slokh merged 4 commits intomainfrom
feature/ci-gates

Conversation

@Slokh
Copy link
Collaborator

@Slokh Slokh commented Feb 26, 2026

This PR now keeps CI minimal: tests-only.

  • Removed coverage (cargo-llvm-cov) and security (cargo-audit, cargo-deny) workflows.
  • Left the optional local make coverage target for developers; happy to remove it if you prefer.
  • No product code changes.

Rationale: simplify CI until we lock agent-facing contracts; tests are sufficient signal for now.

Slokh and others added 2 commits February 26, 2026 11:45
…(cargo-audit, cargo-deny); add informational MSRV job; add local make coverage target

Amp-Thread-ID: https://ampcode.com/threads/T-019c9ab7-3294-76f8-bcbd-2edc07323c50
Co-authored-by: Amp <amp@ampcode.com>
@github-actions
Copy link
Contributor

github-actions bot commented Feb 26, 2026

Tempo Lint Results

Summary

Found 103 issue(s) across 10 file(s)

Severity Count
Errors 0
Warnings 103
Hints 0

Issues by Rule Type

no-unwrap-in-lib (102 occurrences)
  • /home/runner/work/presto/presto/src/network.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/network.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/mod.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.
  • /home/runner/work/presto/presto/src/payment/session/store.rs:1 - Avoid .unwrap() in library code. Use proper error handling with ? or .expect() with context.

... and 92 more

no-leading-whitespace-strings (1 occurrence)
  • /home/runner/work/presto/presto/src/config.rs:1 - String literals should not start with leading whitespace. Found: ' = '

Issues by File

View grouped by file

/home/runner/work/presto/presto/src/payment/session/store.rs (48 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • ... and 43 more

/home/runner/work/presto/presto/src/wallet/credentials/mod.rs (21 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • ... and 16 more

/home/runner/work/presto/presto/src/wallet/keychain.rs (13 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • ... and 8 more

/home/runner/work/presto/presto/src/wallet/signer.rs (7 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • ... and 2 more

/home/runner/work/presto/presto/src/wallet/key_authorization.rs (6 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • ... and 1 more

/home/runner/work/presto/presto/src/http.rs (3 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib

/home/runner/work/presto/presto/src/network.rs (2 issues)

  • Line 1: [warning] no-unwrap-in-lib
  • Line 1: [warning] no-unwrap-in-lib

/home/runner/work/presto/presto/src/payment/session/mod.rs (1 issue)

  • Line 1: [warning] no-unwrap-in-lib

/home/runner/work/presto/presto/src/config.rs (1 issue)

  • Line 1: [warning] no-leading-whitespace-strings

/home/runner/work/presto/presto/src/wallet/credentials/model.rs (1 issue)

  • Line 1: [warning] no-unwrap-in-lib

Posted by https://github.com/tempoxyz/lints

Slokh and others added 2 commits February 26, 2026 11:48
@Slokh Slokh changed the title ci: add coverage (85% gate), audit/deny checks, and informational MSRV ci: keep tests-only; remove coverage and security workflows Feb 26, 2026
@Slokh Slokh merged commit 5d51f4e into main Feb 26, 2026
2 checks passed
@Slokh Slokh deleted the feature/ci-gates branch February 26, 2026 16:53
Slokh added a commit that referenced this pull request Mar 5, 2026
o-az pushed a commit that referenced this pull request Mar 19, 2026
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