-
Notifications
You must be signed in to change notification settings - Fork 51
Register mpq editor #1529
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
Register mpq editor #1529
Conversation
f923c4b to
d536706
Compare
seanshpark
left a comment
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.
LGTM
package.json
Outdated
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.
May I ask why the command name is 'show*', as far as I understand, it should be 'createMPQJson'...?
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.
Ahh. Sorry. I'll change it createFromDefaultExplorer.
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.
It was meant to show command related to mpq.
This commit registers *.mpq.json and showFromDefaultExplorer for mpq editor. ONE-vscode-DCO-1.0-Signed-off-by: s.malakhov <[email protected]>
d536706 to
579e032
Compare
dayo09
left a comment
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.
LGTM :-D
seanshpark
left a comment
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.
LGTM
This commit registers *.mpq.json and showFromDefaultExplorer for mpq editor.
Its correctness is tested in #1511.
Fresh draft: #1511
Previous draft: #1505
Related: #1491
ONE-vscode-DCO-1.0-Signed-off-by: s.malakhov [email protected]