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

[plug-in] Implement inspect configuration #2289

Closed
AndrienkoAleksandr opened this issue Jul 6, 2018 · 9 comments · Fixed by #4153
Closed

[plug-in] Implement inspect configuration #2289

AndrienkoAleksandr opened this issue Jul 6, 2018 · 9 comments · Fixed by #4153
Assignees
Labels
plug-in system issues related to the plug-in system Team: Che-Editors issues regarding the che-editors team

Comments

@AndrienkoAleksandr
Copy link
Contributor

AndrienkoAleksandr commented Jul 6, 2018

Implement inspect configuration:
theia#WorkspaceConfiguration#inspect<T>(section: string):
https://github.com/theia-ide/theia/blob/v0.3.16/packages/plugin/src/theia.d.ts#L2838

https://github.com/theia-ide/theia/blob/master/packages/plugin-ext/src/plugin/preference-registry.ts#L138

@AndrienkoAleksandr AndrienkoAleksandr added plug-in system issues related to the plug-in system Team: Che-Editors issues regarding the che-editors team labels Jul 6, 2018
@AndrienkoAleksandr
Copy link
Contributor Author

Seems related or depends on: #2117

@evidolob
Copy link
Contributor

evidolob commented Nov 2, 2018

Need for VS Code XML extension

@evidolob evidolob changed the title [plug-in] Plugin api: Implement inspect configuration [plug-in] Implement inspect configuration Nov 2, 2018
@akurinnoy
Copy link
Contributor

depends on #3540

@akosyakov
Copy link
Member

@benoitf @evidolob @akurinnoy Could we provide an implementation without support of multi roots for now?

@akurinnoy
Copy link
Contributor

@akosyakov I believe, yes

@akosyakov
Copy link
Member

@akurinnoy Could you look into it this week? If no, please ping me i may have time to look at it tomorrow.

@akurinnoy
Copy link
Contributor

@akosyakov I've planned to look into it tomorrow. But I don't think I will have time to work on implementation because I'm going to have PTO since next Tuesday and I need to finalize some work till then.

@akurinnoy
Copy link
Contributor

This issue is in our sprint plan and someone else could handle it.
@evidolob @AndrienkoAleksandr WDYT?

@evidolob
Copy link
Contributor

Yes, I can handle it

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
plug-in system issues related to the plug-in system Team: Che-Editors issues regarding the che-editors team
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants