Skip to content

docs: fix cross-chain airdrop README links#4205

Merged
Scottcjn merged 1 commit intoScottcjn:mainfrom
maelrx:docs/fix-cross-chain-airdrop-links
May 10, 2026
Merged

docs: fix cross-chain airdrop README links#4205
Scottcjn merged 1 commit intoScottcjn:mainfrom
maelrx:docs/fix-cross-chain-airdrop-links

Conversation

@maelrx
Copy link
Copy Markdown
Contributor

@maelrx maelrx commented May 10, 2026

Addresses Scottcjn/rustchain-bounties#2178

Summary:

  • Fixed relative links in cross-chain-airdrop/README.md that used ../../ and resolved outside the repository root.
  • Updated the license and contributing links so readers land on valid targets.
  • Kept the change scoped to documentation link targets only.

Verification:

  • Ran a local Markdown link check for cross-chain-airdrop/README.md; all local Markdown links now resolve.

Notes:

  • No payment or private user details are included in this PR.

@github-actions
Copy link
Copy Markdown
Contributor

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!

@maelrx
Copy link
Copy Markdown
Contributor Author

maelrx commented May 10, 2026

I do not have repository label permissions, so I could not add the requested BCOS-L1/BCOS-L2 label myself. The PR is docs-only and currently has the documentation and size/XS labels.

Copy link
Copy Markdown
Contributor

@cerredz cerredz left a comment

Choose a reason for hiding this comment

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

Docs-link review.

I checked the changed paths from cross-chain-airdrop/README.md against the repository root. The four local documentation links now resolve, and the previous LICENSE-APACHE / LICENSE-MIT files do not exist in this repo. Linking Apache text to the existing root LICENSE plus using the public MIT license URL is a reasonable docs-only repair for the current tree.

Validation performed from the repository root:

  • Test-Path .\docs\RIP-305-cross-chain-airdrop.md => true
  • Test-Path .\bridge\README.md => true
  • Test-Path .\rips\docs\RIP-0305-solana-spl-token-deployment.md => true
  • Test-Path .\airdrop\README.md => true
  • Test-Path .\LICENSE and Test-Path .\CONTRIBUTING.md => true
  • Test-Path .\LICENSE-APACHE and Test-Path .\LICENSE-MIT => false

No blocking findings from my review.

@Scottcjn
Copy link
Copy Markdown
Owner

💰 PAID — 2 RTC pending, will confirm in 24h.

  • tx hash: f41f9ec55d958a28eab95b67316c3a3b
  • Pending ID: 1407

What worked

Welcome @maelrx! Broken relative links fixed + license targets aligned with crate metadata (MIT OR Apache-2.0). Trivial tier but always paid — clean docs work earns its faucet RTC.

— auto-triage 2026-05-10

@Scottcjn Scottcjn merged commit 38e474a into Scottcjn:main May 10, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation size/XS PR: 1-10 lines

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants