posit-dev / posit-dev/connectapi
Move uuid to Suggests
Open
Beginner friendly
Nobody has claimed this yet.
- Dominant language
- R
- Stars
- 54
- Forks
- 27
- Avg merge
- 1d 3m
- Merged PRs (30d)
- 1
Description
In the code it's only used as a default value for the name to content_ensure(), which is never called without specifying name. It's used in tests, which seems fine, but it seems safe to move to Suggests instead of Imports.
Contributor guide
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 by locating the package dependency declarations and the definition or callers of content_ensure(). Confirm that uuid is only used for the default name value and in tests; done means moving uuid from Imports to Suggests without breaking the existing test suite.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- build-system
- Issue type
- Refactor
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Activity status
- Stale
- Clarity
- Clearly specified
- Newbie friendliness
- 62/100