Skip to content

feat: add miner_online_checker#1318

Closed
CelebrityPunks wants to merge 1 commit intoScottcjn:mainfrom
CelebrityPunks:feat/miner_online_checker
Closed

feat: add miner_online_checker#1318
CelebrityPunks wants to merge 1 commit intoScottcjn:mainfrom
CelebrityPunks:feat/miner_online_checker

Conversation

@CelebrityPunks
Copy link
Contributor

Check if specific miner is online

@github-actions github-actions bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) miner Miner client related size/S PR: 11-50 lines labels Mar 15, 2026
@github-actions
Copy link

Welcome to RustChain! Thanks for your first pull request.

Before we review, please make sure:

  • Your PR has a BCOS-L1 or BCOS-L2 label
  • New code files include an SPDX license header
  • You've tested your changes against the live node

Bounty tiers: Micro (1-10 RTC) | Standard (20-50) | Major (75-100) | Critical (100-150)

A maintainer will review your PR soon. Thanks for contributing!

Copy link

@achievefibromyalgia-lgtm achievefibromyalgia-lgtm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review: #1318

Verdict: ✅ Approved!

@Scottcjn
Copy link
Owner

Hey @CelebrityPunks — want to be upfront with you.

Your earlier work was genuinely excellent: the security audit (#990), CLI wallet (#991), block explorer (#1002), TUI dashboard (#1001) — that's real engineering and you earned every RTC of the 129 we paid.

But the recent batch of ~100 PRs (#1318#1427) are all the same 20-line template with different filenames. wallet_sweep_dust.py doesn't sweep dust — it prints "Sweep small balances to main" and checks node health. Same code in every file.

We can't pay bounties for copy-pasted templates. That said, you're still a valued contributor with a track record of real work. If you want to pick up a bounty, the bounty board has real tasks at 3-15 RTC each. Your skills are clearly there — the security audit proved it.

Quality over quantity. One real tool > 100 template scripts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) miner Miner client related size/S PR: 11-50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants