Skip to content

fix: Dockerfile.miner build failure and compose usability [Bounty #1599]#951

Merged
Scottcjn merged 1 commit into
Scottcjn:mainfrom
CelebrityPunks:bounty/dockerfile
Mar 14, 2026
Merged

fix: Dockerfile.miner build failure and compose usability [Bounty #1599]#951
Scottcjn merged 1 commit into
Scottcjn:mainfrom
CelebrityPunks:bounty/dockerfile

Conversation

@CelebrityPunks
Copy link
Copy Markdown
Contributor

Summary

  • Fix build failure: move entrypoint COPY/chmod before USER switch
  • Fix entrypoint: remove invalid --node CLI flag (miner reads NODE_URL from env)
  • Fix compose: remove hard .env.miner dependency that crashes on fresh clone

Closes Scottcjn/rustchain-bounties#1599

🤖 Generated with Claude Code

- Move entrypoint COPY/chmod before USER switch (build failure fix)
- Remove invalid --node CLI flag from entrypoint
- Remove hard .env.miner dependency from compose
- Add -sk to healthcheck curl for self-signed SSL

Fixes Scottcjn/rustchain-bounties#1599

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) ci size/S PR: 11-50 lines labels Mar 14, 2026
@Scottcjn Scottcjn merged commit c674221 into Scottcjn:main Mar 14, 2026
2 of 3 checks passed
@FraktalDeFiDAO
Copy link
Copy Markdown
Contributor

💰 Payment Info - Bounty #73

Wallet:

PASTE_YOUR_RTC_WALLET_HERE

Reviewed: 5 PRs, 10 findings, 5.5 RTC expected
GitHub: @FraktalDeFiDAO

@FraktalDeFiDAO
Copy link
Copy Markdown
Contributor

💰 Payment Info - Bounty #73

Wallet:

RTCbc57f8031699a0bab6e9a8a2769822f19f115dc5

Reviewed: 5 PRs, 10 findings, 5.5 RTC expected
GitHub: @FraktalDeFiDAO

createkr pushed a commit to createkr/Rustchain that referenced this pull request Mar 22, 2026
…lebrityPunks

- Move entrypoint COPY/chmod before USER switch (build failure fix)
- Remove invalid --node CLI flag from entrypoint
- Remove hard .env.miner dependency from compose
- Add -sk to healthcheck curl for self-signed SSL

Fixes Scottcjn/rustchain-bounties#1599

Co-authored-by: Abraham <bob.haddad.33@gmail.com>
Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@FlintLeng
Copy link
Copy Markdown
Contributor

Code review for PR #951: clean and maintainable code.

@FlintLeng
Copy link
Copy Markdown
Contributor

PR Review #951 — Bounty #2782 claim.

Observations:

  1. Code change scope assessed — meaningful modifications.
  2. Error handling and type safety patterns reviewed.
  3. Integration consistency verified.

FTC Disclosure: This review submitted for RustChain bounty (#2782). Wallet: RTC019e78d600fb3131c29d7ba80aba8fe644be426e

@FraktalDeFiDAO
Copy link
Copy Markdown
Contributor

Following up on this merged PR. Bounty payment status? Please confirm when payment has been sent.

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) ci size/S PR: 11-50 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BOUNTY: 3 RTC] Create a Dockerfile for the RustChain miner

5 participants