Jonas Helming
Jonas Helming
@colin-grant-work It would help us a lot, if this could make it to the upcoming release. So if you have some cycles, we would be very thankful for a review....
@vince-fugnitto @colin-grant-work @msujew can we merge this? We should probably merge this some days before the release.
@msujew : Would you mind looking at this soon, I do not want to merge this too late in the dev cycle.
Just out of interest: Did you consider moving this to CDT.cloud instead of the Theia main repo?
Basically to keep the core Theia project as slim as possible. Most components would benefit from beeing moved to the core project/repo in terms of ensuring compatibility. Are the people...
Thinking about this again: the memory inspector is not necessarily bound to C/C++, correct? This would be an argument in favor to keep it in Theia.
@colin-grant-work Is this ready for a re-review?
@paul-marechal Can you confirm this is fixed?
@federicobozzini Are you interested in addressing @colin-grant-work comment above?
@yacosdad Are you interested in addressing the review comments?