-
-
Notifications
You must be signed in to change notification settings - Fork 121
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
inheritence and version overrides #3
Comments
Yeah, the problem here is an overarching module/parameter design issue. We're strapped for time but definitely agree, this needs an overhaul. |
This issue hasn't had any updates in nearly 3 years and nothing seems to refer to a remaining problem so I'm going to mark this as closed... if there is still an issue around this please feel free to re-open and add more up to date details.... thanks. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Heads up... I've not yet tracked down all of these issues here but if you classify as such:
you won't get the version specified due to the way the default version is specified in params and set in the implementation class via inheritance. I believe that the use of way inheritance is used in this module may mean there are other similar issues.
The text was updated successfully, but these errors were encountered: