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

remove unused courses from PathConfig #1702

Closed
jurgenvinju opened this issue Nov 1, 2022 · 1 comment
Closed

remove unused courses from PathConfig #1702

jurgenvinju opened this issue Nov 1, 2022 · 1 comment
Labels
enhancement

Comments

@jurgenvinju
Copy link
Member

Describe the bug

The list[loc] courses = [] field of util::Reflective::PathConfig was used to configure the tutor compiler. The current tutor compiler uses the list[loc] srcs = [] field to do the same, as if it were just any other compiler.

Removing this field in PathConfig.java and util::Reflective will take some doing, but it will clean up the interface and remove a few dozen unused lines of code.
@DavyLandman
Copy link
Member

I saw a PR sneak past with these changes?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@DavyLandman @jurgenvinju