-
Notifications
You must be signed in to change notification settings - Fork 3
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
Revoke namespace change #29
Revoke namespace change #29
Conversation
I've also deleted the ProcessSetup tests since fixing them started taking too long and they are deprecated anyways. |
63693ed
to
e100971
Compare
General question since we will now often implement interfaces defined somewhere else (i.e. in QEDbase): I think for clarity it would be a good idea to always use I've done it using Thoughts? |
As I mentioned somewhere else, I think |
@AntonReinhard is there a reason, that this is still a draft? |
Yes, as I mentioned I will go through again and remove the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LTFM.
This PR revokes the namespace changes done before QEDbase.jl 0.2 was released.
Currently based on #25, I will rebase once that is merged.