Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Consolidating Script Formatting Functions #9345

Open
Patowhiz opened this issue Jan 10, 2025 · 1 comment
Open

Consolidating Script Formatting Functions #9345

Patowhiz opened this issue Jan 10, 2025 · 1 comment

Comments

@Patowhiz
Copy link
Contributor

Currently, we have multiple functions responsible for formatting scripts for various purposes, such as inspecting script contents or displaying them in the script or output windows. These functionalities have significant overlap but are implemented across different classes.

To improve maintainability and adhere to the DRY (Don't Repeat Yourself) principle, I propose consolidating these implementations into a single class or a cohesive set of related classes. This will streamline the functionality and make the codebase easier to manage and extend.

@rdstern
Copy link
Collaborator

rdstern commented Jan 10, 2025

Thanks @Patowhiz , I am adding @lilyclements into this thread. We now have documentation on the R-Instat "packages", i.e. on all the special functions we have been producing. (Note the "Royal we" here, because I haven't produced any myself.)
We have a plan to make use of the packages - rather than each individual function separately - as soon as possible. Your suggestion is relevant here.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants