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

Refactor/split into multiple files #17

Open
clehner opened this issue Jan 9, 2015 · 2 comments
Open

Refactor/split into multiple files #17

clehner opened this issue Jan 9, 2015 · 2 comments

Comments

@clehner
Copy link
Member

clehner commented Jan 9, 2015

As the script is now >1000 lines and embracing several areas of functionality, @jeremywrnr and I were discussing splitting it into separate files. We propose keeping core functions (like get courses) and command line argument parsing in bb while making one file for each command (except commands where there is significant overlap).

This could be done in such a way that only the scripts which need to be executed are sourced. e.g.

case "$cmd" in
...
bill) source $BB_PATH/quikpay; bb_bill $@;;
...
@jthurst3
Copy link
Contributor

jthurst3 commented Jan 9, 2015

I was going to suggest the same thing. Sounds like a great idea to me.

@jeremywrnr
Copy link
Contributor

import.sh repo here is interesting and may be useful for dealing with this issue.

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

No branches or pull requests

3 participants