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

Add term as a folder. #56

Open
wants to merge 60 commits into
base: master
Choose a base branch
from
Open

Add term as a folder. #56

wants to merge 60 commits into from

Conversation

wznmickey
Copy link
Contributor

Do #52

@wznmickey wznmickey changed the title Add term as a folder. WIP: Add term as a folder. Sep 8, 2023
@wznmickey wznmickey changed the title WIP: Add term as a folder. Add term as a folder. Sep 8, 2023
@BoYanZh
Copy link
Owner

BoYanZh commented Sep 23, 2023

Why not just use a separate config file for each term?

@wznmickey
Copy link
Contributor Author

Some courses like undergraduate are not in the same term as regular courses. So the user may need to switch config files and rerun the script many times to download new files. The work of user will increase. Besides, the config file in hidden in the package folder, so some users may not find the place.

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

Successfully merging this pull request may close these issues.

2 participants