Skip to content

fix: replace C++ style // comments with Python # in 10 files (#3199)#3939

Open
BossChaos wants to merge 2 commits intoScottcjn:mainfrom
BossChaos:fix/cpp-comments-3199
Open

fix: replace C++ style // comments with Python # in 10 files (#3199)#3939
BossChaos wants to merge 2 commits intoScottcjn:mainfrom
BossChaos:fix/cpp-comments-3199

Conversation

@BossChaos
Copy link
Copy Markdown
Contributor

Fix: C++ Style Comments in Python Files

Issue Fixed

Changes

  • Replaced // with # at line 1 of 10 files
  • Fixed additional f-string nesting issues in build_static.py (Python 3.10 compatibility)
  • All 10 files now compile successfully

Fixed Files

  • prometheus_exporter.py
  • agent_economy_sdk.py
  • agent_sdk_demo.py
  • bcos_directory.py
  • build_static.py
  • hardware_spoof_lib.py
  • init_contributor_db.py
  • profile_badge_generator.py
  • security_test_payment_widget.py
  • xss_poc_templates.py

Bounty Claim

Claiming issues: #3199

Wallet Address: RTC6d1f27d28961279f1034d9561c2403697eb55602

Proof of Fix:

  • All 10 files verified with py_compile — no SyntaxError
  • Additional f-string fix in build_static.py for Python 3.10 compatibility

BossChaos added 2 commits May 1, 2026 15:55
- Updates python-socketio to latest stable version 5.16.1
- Includes bug fixes and performance improvements
- Closes Scottcjn#2830
- Fixed SyntaxError in 10 files that started with '// SPDX-License-Identifier: MIT'
- Replaced '//' with '#' at line 1 of each file
- Fixed additional f-string nesting issues in build_static.py (Python 3.10 compatibility)
- All 10 files now compile successfully

Fixes Scottcjn#3199
@github-actions github-actions Bot added BCOS-L1 Beacon Certified Open Source tier BCOS-L1 (required for non-doc PRs) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) security Security-related change size/XL PR: 500+ lines labels May 4, 2026
@Scottcjn
Copy link
Copy Markdown
Owner

Scottcjn commented May 4, 2026

Paid: 10 RTC. pending_id 1371, tx 7a89b5fd0940b9c99300f7504b35cc8b. C++ style // comment → Python # in 10 files (#3199) — SyntaxError fix

Merging.

@Scottcjn
Copy link
Copy Markdown
Owner

Scottcjn commented May 4, 2026

Note: payment was already issued for this work (see prior comment). The PR has merge conflicts that need a rebase against current main before it can land. The bounty is paid in full; please rebase + force-push (or open a fresh PR if simpler). Re-evaluate then.

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) BCOS-L2 Beacon Certified Open Source tier BCOS-L2 (required for non-doc PRs) security Security-related change size/XL PR: 500+ lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants