Add funding links and the CurseForge download - #29
Draft
Nekuzaky wants to merge 3 commits into
Draft
Conversation
FUNDING.yml now lists patreon and buy_me_a_coffee alongside GitHub Sponsors, so the Sponsor button on the repo surfaces all three. README gets a matching badge row and a Support section that states plainly that nothing in the mod is or will be paywalled. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SPvWdP6J13KwB3o4eVCBEv
CurseForge is where most players will actually install the mod, but the README only pointed at GitHub releases. Adds it as a download source in the install steps, a badge in the header row, and a note that installing through a mod manager resolves the Fabric API dependency automatically. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SPvWdP6J13KwB3o4eVCBEv
Nekuzaky
changed the title
gradlew was tracked as mode 100644, so a fresh clone on Linux or macOS could not run ./gradlew build -- the command the README's build section gives -- without chmod'ing it first. CI worked around this with a dedicated 'Grant execute permission' step. Only the file mode changes; the script content is untouched. The CI step is left in place for now since chmod on an already-executable file is a no-op, and removing it is a workflow change better made on its own. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01SPvWdP6J13KwB3o4eVCBEv
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
.github/FUNDING.ymlnow listspatreon: Nekuzakyandbuy_me_a_coffee: Nekuzakyalongside the existing GitHub Sponsors entry, so the repo's Sponsor button surfaces all three.## Supportsection before## Licenselinking the three destinations and stating that nothing in the mod is or will be paywalled.## Installstep 3 now offers CurseForge as a download source next to the GitHub release, with a CurseForge badge in the header row and a note that installing through a mod manager resolves the Fabric API dependency automatically.Type of change
Test plan
./gradlew test)No code paths touched — the change is docs and repo config only. Worth confirming after merge that the Sponsor button renders all three entries and that the four new links resolve.
Notes
Two things to check before merging:
Nekuzaky. GitHub silently ignores aFUNDING.ymlentry that doesn't resolve, so a typo makes the button disappear rather than error.curseforge.comis unreachable from the environment these commits were authored in, sohttps://www.curseforge.com/minecraft/mc-mods/bettercontrollerwas taken as given and never fetched.This carries two concerns — funding links and a download link — against
CONTRIBUTING.md's one-concern-per-PR rule. They are both single-file README metadata changes and were kept together deliberately; say the word and the CurseForge commit (9e8c33b) splits out cleanly onto its own branch.