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 the project layout, split into modules by functionality #12

Merged
merged 4 commits into from
Jul 22, 2024
Merged

Refactor the project layout, split into modules by functionality #12

merged 4 commits into from
Jul 22, 2024

Conversation

seapagan
Copy link
Owner

  • move CLI code out to a module

@seapagan
move CLI code out to a module
0f72bee 
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
@seapagan seapagan added the refactor Code refactoring label Jul 22, 2024
@seapagan seapagan self-assigned this Jul 22, 2024
seapagan added 3 commits July 22, 2024 08:55
@seapagan
use a struct for 'collect_file_names' params
1485068 
cuts down number of paramters to pass

Signed-off-by: Grant Ramsay <seapagan@gmail.com>
@seapagan
rename 'CLI' structure to 'Flags'
33b8c9d 
gives a better layout and clearer use

Signed-off-by: Grant Ramsay <seapagan@gmail.com>
@seapagan
break 'utils' functionality into multiple modules
1d9025b 
Signed-off-by: Grant Ramsay <seapagan@gmail.com>
@seapagan seapagan merged commit 82f2532 into main Jul 22, 2024
1 check passed
@seapagan seapagan deleted the refactor-layout branch July 22, 2024 08:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@seapagan seapagan

Labels
refactor Code refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@seapagan