popcodeorg / popcodeorg/popcode
Don’t suggest enabling p5 based on global usage
Nobody has claimed this yet.
- Dominant language
- JavaScript
- Stars
- 191
- Forks
- 143
- Avg merge
- 4d 11h
- Merged PRs (30d)
- 5
Description
P5 defines quite a few global variables and functions, many of which have common words for names. Suggesting enabling P5 every time someone references an undefined variable like day is unlikely to be the most helpful suggestion.
Practical suggestion: only auto-suggest library globals if the library defines three or fewer globals in total.
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start with src/config/libraries.js, especially the p5 global definitions linked in the issue, then locate the code that generates suggestions for undefined variables. Verify the current behavior for a common name such as day; done means libraries with more than three globals are no longer automatically suggested while smaller libraries remain eligible.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- javascript
- Domain
- tooling
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 38/100