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

create endpoint /reloadsolution #19

Open
nosami opened this issue Dec 14, 2014 · 8 comments
Open

create endpoint /reloadsolution #19

nosami opened this issue Dec 14, 2014 · 8 comments

Comments

@nosami
Copy link
Contributor

nosami commented Dec 14, 2014

Reloads the current Workspace with the files on disk. Useful after making a big change to the solution such as deleting a folder or switching git branches etc.

@cubitouch
Copy link
Contributor

Wanna help on that.
I understand what you need, but is there already an endpoint doing something similar ?

@nosami
Copy link
Contributor Author

nosami commented Feb 22, 2015

There isn't an endpoint for this yet.

It should mimic what the old endpoint
https://github.com/OmniSharp/omnisharp-server/blob/master/OmniSharp/ReloadSolution/ReloadSolutionModule.cs#L10
in omnisharp-server did, i.e. reinitialise the workspace and projects.

It would be useful when switching branch during a git checkout for example

  • any time when the solution substantially changes.

On 22 Feb 2015 08:37, "Hugo CARNICELLI" notifications@github.com wrote:

Wanna help on that.
I understand what you need, but is there already an endpoint doing
something similar ?


Reply to this email directly or view it on GitHub
#19 (comment)
.

@cubitouch
Copy link
Contributor

Thanks, I will work on that this week :)

@nosami
Copy link
Contributor Author

nosami commented Feb 22, 2015

👍

@cubitouch
Copy link
Contributor

Should this return a response ?

@nosami
Copy link
Contributor Author

nosami commented Feb 24, 2015

It should just return true if successful.

@david-driscoll david-driscoll changed the title /reloadsolution create endpoint /reloadsolution Mar 22, 2016
@TheRealPiotrP
Copy link

@DustinCampbell seems this was fixed in #101. Can we close this?

@DustinCampbell
Copy link
Contributor

AFAICT, #101 was never merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants