docs(credits): credit marshmallow-mia's six merged PRs, and the co-author line #1569's merge left out

Open Beginner friendly
  • Is something wrong?
#2,137 0 comments 0 reactions 0 assignees View on GitHub

Maintainers usually reply within 1 day

Nobody has claimed this yet.

Assessment

Difficulty
2/5
Estimated time
1-3 hours
Newbie friendliness
74/100
Issue type
Documentation
Clarity
Clearly specified
Activity status
Active
Tech stack
cpp, json
Domain
documentation

Research direction

Start with the marshmallow-mia entry in config/credits.json and add the six merged PRs listed in the issue. Check how the README and in-game credits render that file, then run tools/tests/test_credits.py. The separate PR should include a Co-Authored-By trailer for her and explain that the line is owed from #1569.

Written by the indexing model from the issue text.

Description

documentation

config/credits.json credits marshmallow-mia for work outside this repository, but not for the PRs she wrote here. One later merge on the same weapons carries no co-author line for her.

Evidence at 0775d335

  • config/credits.json, the marshmallow-mia entry: its what names the Linux port, the Echo VR installer, EchoVR_Launcher and NvrContentOverlay. It names none of her six PRs merged here between 2026-10-29 and 2026-10-30: #1442 (a plugin serves a content pack), #1450 (render_config carries the GPU score), #1453 (a texture's payload replaced by an entry), #1480 (the magnum, smg, Chain and rifle bullets), #1491 (network.max_team_size), #1495 (network.bots_per_session).
  • #1569, merged as 64e67852, changes the netgun bullets of the same four unfinished weapons that #1480 changed. Its Co-Authored-By trailers name two co-authors, neither of them her.

Not shown

  • Which lines of #1569 draw on #1480.

Falsifier

  • A credits entry, or a later commit's trailer, that already credits her for these PRs and for #1569.

Work

  1. Add the six PRs to her config/credits.json entry. The README and in-game credits render from that file, so check that both show them.
  2. A pushed commit isn't rewritten, so 64e67852 keeps its trailers. The PR that fixes this carries Co-Authored-By for her, and its body says the line is owed from #1569.
  3. The credits tests pass (tools/tests/test_credits.py).

꩜🌸

Dominant language
C++
Stars
2
Forks
5
Avg merge
4h 20m
Merged PRs (30d)
939

Getting set up

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

More from EchoTools/nevr-runtime

All issues in EchoTools/nevr-runtime

Similar issues

More C++ issues

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.