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 the AlgorithmProxy class #28407

Closed
NickDraper opened this issue Mar 20, 2020 · 0 comments · Fixed by #28409
Closed

Remove the AlgorithmProxy class #28407

NickDraper opened this issue Mar 20, 2020 · 0 comments · Fixed by #28409
Assignees
@NickDraper
Labels
ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Milestone
Release 5.1 S1

Comments

@NickDraper
Copy link
Contributor

The Proxy class should no longer be needed, let's remove it.

Leave IAlgorithm in place though.
@NickDraper NickDraper added the ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS label Mar 20, 2020
@NickDraper NickDraper added this to the Release 5.1 S1 milestone Mar 20, 2020
@NickDraper NickDraper self-assigned this Mar 20, 2020
@NickDraper NickDraper mentioned this issue Mar 20, 2020
Merged
@martyngigg martyngigg mentioned this issue Aug 7, 2020
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@NickDraper NickDraper

Labels
ISIS Team: Core Issue and pull requests managed by the Core subteam at ISIS
Projects
None yet
Milestone
Release 5.1 S1
Development

Successfully merging a pull request may close this issue.

Remove algorithm proxy class mantidproject/mantid
1 participant
@NickDraper