Skip to content

feat(nano): add check-blueprint CLI#1315

Merged
glevco merged 1 commit intomasterfrom
feat/check-blueprint
Jul 11, 2025
Merged

feat(nano): add check-blueprint CLI#1315
glevco merged 1 commit intomasterfrom
feat/check-blueprint

Conversation

@glevco
Copy link
Contributor

@glevco glevco commented Jul 10, 2025

Motivation

Implement a new check-blueprint CLI command for Blueprint devs to check the syntax validity of their Blueprints.

Usage:

poetry run hathor-cli check-blueprint --file my_blueprint.py

Acceptance Criteria

  • Add new check-blueprint CLI command.
  • Refactor OCB verifier to create a single verify_code method.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@glevco glevco requested a review from pedroferreira1 July 10, 2025 20:32
@glevco glevco self-assigned this Jul 10, 2025
@glevco glevco requested review from jansegre and msbrogli as code owners July 10, 2025 20:32
@glevco glevco force-pushed the feat/check-blueprint branch from f749a57 to 8cfcf43 Compare July 10, 2025 20:34
@glevco glevco moved this from Todo to In Progress (Done) in Hathor Network Jul 10, 2025
@github-actions
Copy link

github-actions bot commented Jul 10, 2025

🐰 Bencher Report

Branchfeat/check-blueprint
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
1.63 m
(-0.52%)Baseline: 1.64 m
1.47 m
(90.47%)
1.80 m
(90.44%)
🐰 View full continuous benchmarking report in Bencher

@github-project-automation github-project-automation bot moved this from In Progress (Done) to In Review (WIP) in Hathor Network Jul 10, 2025
@glevco glevco moved this from In Review (WIP) to In Review (Done) in Hathor Network Jul 10, 2025
@codecov
Copy link

codecov bot commented Jul 10, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 85.58%. Comparing base (e417e0e) to head (c7643e0).
Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1315      +/-   ##
==========================================
- Coverage   85.63%   85.58%   -0.05%     
==========================================
  Files         426      426              
  Lines       31947    31949       +2     
  Branches     4962     4962              
==========================================
- Hits        27358    27344      -14     
- Misses       3590     3604      +14     
- Partials      999     1001       +2     

☔ 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.

@glevco glevco force-pushed the feat/check-blueprint branch from 8cfcf43 to c7643e0 Compare July 10, 2025 23:04
@glevco glevco merged commit 7963762 into master Jul 11, 2025
8 checks passed
@glevco glevco deleted the feat/check-blueprint branch July 11, 2025 02:02
@github-project-automation github-project-automation bot moved this from In Review (Done) to Waiting to be deployed in Hathor Network Jul 11, 2025
@jansegre jansegre mentioned this pull request Jul 22, 2025
2 tasks
@jansegre jansegre moved this from Waiting to be deployed to Done in Hathor Network Jul 22, 2025
@jansegre jansegre mentioned this pull request Aug 7, 2025
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants