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

class MV is now broken #27

Closed
3 tasks done
utensil opened this issue Mar 22, 2019 · 0 comments · Fixed by #17
Closed
3 tasks done

class MV is now broken #27

utensil opened this issue Mar 22, 2019 · 0 comments · Fixed by #17

Comments

@utensil
Copy link
Member

utensil commented Mar 22, 2019

Due to my commit Remove circular dependencies , module mv no longer depends on module ga, but currently MV still lives in mv and calls ga.

So any examples using class MV, particularly the ones in examples/Old Format won't work and that includes some references to class MV in the new README. (My mistake......)

I have no intention to reintroduce circular dependencies or to drop support for class MV, so here's the work needed to be done:

  • Create a new module that depends on ga and mv to put class MV and possibly the widely used com(), could name it legacy or deprecated (related to a179957)
  • Fix any references to class MV in README or doc
  • Fix examples using class MV
@utensil utensil mentioned this issue Mar 22, 2019
19 tasks
@utensil utensil added this to the 0.4.4 milestone Apr 3, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant