insightsengineering / insightsengineering/teal.goshawk

Deprecate `filter_var` argument in `tm_g_gh_spaghettiplot`

Open
#316 0 comments 0 reactions 0 assignees View on GitHub
core good first issue TrackA_RMedicine_Hackathon_Apr2026
Dominant language
R
Stars
3
Forks
3
Avg merge
2d 6h
Merged PRs (30d)
11

Description

> I agree. Also, I don't think we need `filter_var` parameter for `tm_g_gh_spaghettiplot` since it's not being used at all in the code. We should remove this.

_Originally posted by @donyunardi in https://github.com/insightsengineering/teal.goshawk/issues/301#issuecomment-2397271516_

### Summary
`filter_var` argument is not being used anywhere in the function body of `tm_g_gh_spaghettiplot`.

### Acceptance Criteria
* Confirm that this is no longer being used.
* Deprecate `filter_var` argument using lifecycle
* Match with other teal packages on how we normally deprecate feature.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.