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.

Sign up for GitHub

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

Jump to bottom

Refactor scripts #58

Open
katyhuff opened this issue Feb 11, 2018 · 2 comments
Open

Refactor scripts #58

katyhuff opened this issue Feb 11, 2018 · 2 comments
Assignees
@KennellyT
Labels
Comp:Analysis Difficulty:2-Challenging Priority:2-Normal Status:1-New Type:Bug

Comments

@katyhuff
Copy link
Member

All scripts in the scripts folder need to be reviewed and to be refactored for clarity.

This issue can be closed when all scripts have been refactored to have smaller functions, specifically, functions small enough to unit test.
@katyhuff katyhuff mentioned this issue Feb 11, 2018
Closed
@jbae11
Copy link
Collaborator

jbae11 commented Mar 27, 2018

closed by #47

@jbae11 jbae11 closed this as completed Mar 27, 2018
@katyhuff
Copy link
Member Author

What you say is impossible. This issue was opened on Feb 11 2018. PR #47 was merged in August 2017.

@katyhuff katyhuff reopened this Mar 27, 2018
@KennellyT KennellyT self-assigned this Jul 4, 2018
@nsryan2 nsryan2 mentioned this issue Aug 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@KennellyT KennellyT

Labels
Comp:Analysis Difficulty:2-Challenging Priority:2-Normal Status:1-New Type:Bug
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@katyhuff @jbae11 @gwenchee @KennellyT