Skip to content

Feat/gambarewards branding#14

Open
4eckd wants to merge 19 commits intogambarewards/ignition2from
feat/gambarewards-branding
Open

Feat/gambarewards branding#14
4eckd wants to merge 19 commits intogambarewards/ignition2from
feat/gambarewards-branding

Conversation

@4eckd
Copy link
Copy Markdown

@4eckd 4eckd commented Mar 18, 2026

Issue # (if available)

Description of changes

Checklist

  • Added change to the changelog
  • Created unit tests for my feature (if needed)
  • Created a least one integration test

claude and others added 3 commits March 18, 2026 10:36
…() preset

- Add gambarewards.com as a fourth Fused Gaming target domain across all
  documentation, examples, and source references
- Introduce IgnitionProfile::gambarewards() preset (Windows + RTX 4080 +
  16 GB + 12 cores) specifically optimised for gambarewards.com
- Update fused_gaming() preset docs to include gambarewards.com
- Add gambarewards.com to TARGETS in examples/fused_gaming.rs

https://claude.ai/code/session_01FuHdSY9g1FRVhndraAmc69
…PWFix

feat: add gambarewards.com branding and IgnitionProfile::gambarewards…
…tically incorrect with two "and"s instead of one

This commit fixes the issue reported at src/lib.rs:7

**Bug Details:**
The documentation comment in src/lib.rs (lines 5-8) contained a grammatical error with two instances of "and" in a comma-separated list. The original text read:
```
Designed for use with [stakereload.com](https://stakereload.com),
[stakereloadxs.com](https://stakereloadxs.com), and
[gambareload.com](https://gambareload.com), and
[gambarewards.com](https://gambarewards.com).
```

This violates proper comma-separated list grammar, where "and" should only appear before the final item. The impact is purely cosmetic - affecting documentation readability and appearing unprofessional.

**Fix Applied:**
Removed the second "and" from line 7 and adjusted the punctuation so that only one "and" appears before the final item:
```
Designed for use with [stakereload.com](https://stakereload.com),
[stakereloadxs.com](https://stakereloadxs.com),
[gambareload.com](https://gambareload.com), and
[gambarewards.com](https://gambarewards.com).
```

This follows the Oxford comma convention for properly formatted lists in English.

Co-authored-by: Vercel <vercel[bot]@users.noreply.github.com>
Co-authored-by: 4eckd <playxrewards@gmail.com>
@vercel
Copy link
Copy Markdown

vercel bot commented Mar 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
gambareload Ready Ready Preview, Comment Mar 18, 2026 0:30am
gambarewards Ready Ready Preview, Comment Mar 18, 2026 0:30am
ignition Ready Ready Preview, Comment Mar 18, 2026 0:30am
stakeclaimbot Ready Ready Preview, Comment Mar 18, 2026 0:30am
stakereload Ready Ready Preview, Comment Mar 18, 2026 0:30am
stakereloadxs Ready Ready Preview, Comment Mar 18, 2026 0:30am

Request Review

Copy link
Copy Markdown

@vercel vercel bot left a comment

Choose a reason for hiding this comment

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

Additional Suggestion:

Documentation says "all three domains" but README lists 6 domains, and Target Platforms table is missing 2 domains

Fix on Vercel

Comment thread README.md
…s, and Target Platforms table is missing 2 domains

This commit fixes the issue reported at README.md:23

## Bug Explanation
The README.md file contained stale and inconsistent documentation regarding the number of supported domains:

1. **Line 23** in the Features section stated: "Pre-built `IgnitionProfile::fused_gaming()` preset for all three domains"
2. However, the introduction (lines 12-17) explicitly lists **6 domains**: stakereload.com, stakereloadxs.com, gambareload.com, stakeclaimbot.com, gambarewards.com, and hedgedhog.com
3. The **Target Platforms table** (around line 94) was only listing **4 domains**, missing stakeclaimbot.com and hedgedhog.com

This inconsistency creates confusion for users - the intro says 6 domains are supported, the features say 3 (outdated), and the reference table lists only 4. This appears to have been introduced when stakeclaimbot.com and hedgedhog.com were added to the intro list but the documentation wasn't fully updated.

## Fix Applied
1. Updated line 23 from "all three domains" to "all six domains" to match the actual number of domains listed in the introduction
2. Added the two missing domains (stakeclaimbot.com and hedgedhog.com) to the Target Platforms table with appropriate descriptions, bringing the table from 4 entries to 6 entries

This ensures consistency across all three places where domains are mentioned: the introduction list, the feature description, and the reference table.

Co-authored-by: Vercel <vercel[bot]@users.noreply.github.com>
Co-authored-by: 4eckd <playxrewards@gmail.com>
Comment thread CLAUDE.md
Comment thread examples/fused_gaming.rs
Comment thread src/lib.rs Outdated
Comment on lines +7 to +8
//! [gambareload.com](https://gambareload.com), and
//! [gambarewards.com](https://gambarewards.com).
Copy link
Copy Markdown

@vercel vercel bot Mar 18, 2026

Choose a reason for hiding this comment

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

Suggested change
//! [gambareload.com](https://gambareload.com), and
//! [gambarewards.com](https://gambarewards.com).
//! [gambareload.com](https://gambareload.com),
//! [stakeclaimbot.com](https://stakeclaimbot.com),
//! [gambarewards.com](https://gambarewards.com), and
//! [hedgedhog.com](https://hedgedhog.com).

Crate-level documentation in src/lib.rs lists only 4 domains while README.md lists 6 domains, causing inconsistency about officially supported platforms

Fix on Vercel

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

The patch has been successfully applied. The crate-level documentation in src/lib.rs now lists all 6 domains (stakereload.com, stakereloadxs.com, gambareload.com, stakeclaimbot.com, gambarewards.com, and hedgedhog.com) in alignment with README.md, resolving the documentation inconsistency.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

This suggestion has been addressed.

Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Co-authored-by: vercel[bot] <35613825+vercel[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants