The-Strategy-Unit / The-Strategy-Unit/nhp_inputs
Refactor/rethink naming/organisation of modules and functions in R/
Open
Nobody has claimed this yet.
could
techdebt
- Dominant language
- R
- Stars
- 2
- Forks
- 1
- Avg merge
- 15h 39m
- Merged PRs (30d)
- 6
Description
mod_funnel_plot.R and mod_mitigator_rates_boxplot.R should really have the fct_ prefix if this naming convention is continued. Related to #346 as a general repo tidy-up and rethink.
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 by reading R/mod_funnel_plot.R and R/mod_mitigator_rates_boxplot.R, then review the naming convention and related issue #346. Determine whether the repository should consistently use the fct_ prefix and how the broader module and function organisation should be revised. Done means the naming and organisation direction is agreed and applied consistently.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- r
- Domain
- tooling
- Issue type
- Refactor
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 25/100