Quenty / Quenty/NevermoreEngine

Signal and OldSignal have conflated docs

Open
#583 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

bug
Dominant language
Lua
Stars
610
Forks
144
Avg merge
1d 20h
Merged PRs (30d)
12

Description

These classes share an API page as a result of a naming collision. According to their Moonwave comments, Signal.lua defines GoodSignal, and OldSignal.lua defines Signal.

However, "Signal" is the actual class/table name in both files, so their members are aggregated under Signal, and GoodSignal exists as a member-less page.

Additionally, OldSignal attempts to redirect to GoodSignal, which doesn't exist given the current file name.

Contributor guide

Open the contributing guide

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.

Research direction

Inspect the Moonwave comments and class names in Signal.lua and OldSignal.lua first. Confirm how their API pages are generated and how the OldSignal redirect is resolved. Done means Signal and OldSignal have separate API documentation pages, with the documented class names and redirect resolving correctly.

Written by the indexing model from the issue text.

Assessment

Tech stack
lua
Domain
documentation
Issue type
Documentation
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
52/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.