Skip to content

Chore: ci modernize workflows#911

Open
b-l-u-e wants to merge 5 commits intocashubtc:mainfrom
b-l-u-e:chore/ci-modernize-workflows
Open

Chore: ci modernize workflows#911
b-l-u-e wants to merge 5 commits intocashubtc:mainfrom
b-l-u-e:chore/ci-modernize-workflows

Conversation

@b-l-u-e
Copy link
Contributor

@b-l-u-e b-l-u-e commented Mar 4, 2026

I was trying to run Nutshell on a fresh setup and noticed several issues:

  • Outdated and inconsistent GitHub Action versions across workflows
  • Deprecated ::set-output usage producing warnings
  • Duplicated setup steps in the checks workflow
  • Archived chartboost/ruff-action (replaced by astral-sh/ruff-action)
  • Pre-commit hooks triggering deprecation warnings

I addressed these by upgrading workflows, aligning docs with the current setup, and bumping pre-commit hooks.

Changes

Workflow infrastructure

  • Upgrade prepare action (actions-poetry v3, setup-python v5)
  • Refactor checks workflow to use prepare (remove duplicate Poetry/Python setup)
  • Replace archived chartboost/ruff-action with astral-sh/ruff-action

Workflow updates

  • Replace deprecated ::set-output with $GITHUB_OUTPUT in docker.yaml
  • Upgrade actions/checkout to v4 and codecov-action to v5
  • Add least-privilege permissions to all workflows
  • Align poetry-version defaults in regtest workflows
  • Remove duplicate prepare step in Keycloak auth tests

Documentation

  • Align regtest repo and paths with CI (cashu-regtest-enviroment)
  • Add LndRPCWallet to the backend list

Tooling

  • Upgrade pre-commit hooks (including fix for deprecated stage names warning)

Tested

  • Local: make ruff-check && make mypy && make test

@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.76%. Comparing base (89564fe) to head (1418cae).
⚠️ Report is 8 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #911      +/-   ##
==========================================
+ Coverage   61.51%   67.76%   +6.25%     
==========================================
  Files          93       93              
  Lines       11051    11051              
==========================================
+ Hits         6798     7489     +691     
+ Misses       4253     3562     -691     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

b-l-u-e added 5 commits March 7, 2026 07:38
Switch prepare to setup-python v6 with explicit Poetry installation, and refactor checks to reuse the prepare action while aligning defaults to Python 3.10 and Poetry 2.3.2.
- bump workflow actions to current majors
- standardize python defaults to 3.10 in reusable workflows
- standardize poetry defaults to 2.3.2 across CI jobs

Signed-off-by: b-l-u-e <winnie.gitau282@gmail.com>
- normalize python examples to 3.10
- update poetry install version to 2.3.2
- keep Dockerfile poetry version in sync with CI/docs

Signed-off-by: b-l-u-e <winnie.gitau282@gmail.com>
- regenerate lock metadata and normalized markers
- keep dependency graph aligned with current poetry toolchain

Signed-off-by: b-l-u-e <winnie.gitau282@gmail.com>
- pre-commit-hooks: v4.5.0 -> v6.0.0
- ruff-pre-commit: v0.7.1 -> v0.15.4
- mirrors-mypy: v1.6.0 -> v1.19.1

Signed-off-by: b-l-u-e <winnie.gitau282@gmail.com>
@b-l-u-e b-l-u-e force-pushed the chore/ci-modernize-workflows branch from d494d6e to 1418cae Compare March 7, 2026 05:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Backlog

Development

Successfully merging this pull request may close these issues.

1 participant