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

fix(application-package): Fix the return value error of electron.isDevMode API in Windows system in development mode. #10359

Merged

Conversation

zhaomenghuan
Copy link
Contributor

@zhaomenghuan zhaomenghuan commented Nov 1, 2021

What it does

Fix the return value error of electron.isDevMode API in Windows system in development mode.

How to test

Run the Theia source code project based on the Windows system, then open DevTools to execute the following code:

require('@theia/application-package/lib/environment').environment.electron.isDevMode()

image

Review checklist

Reminder for reviewers

Signed-off-by: zhaomenghuan zhaomenghuan02@meituan.com

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @zhaomenghuan, thank you for your contribution, do you mind adding testing steps to your PR?

@msujew msujew added the electron issues related to the electron target label Nov 1, 2021
@zhaomenghuan
Copy link
Contributor Author

zhaomenghuan commented Nov 1, 2021

Hi @zhaomenghuan, thank you for your contribution, do you mind adding testing steps to your PR?

Hi @msujew, I added a test description.

Copy link
Member

@msujew msujew left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can confirm that the issue exists on master and is addressed by this change 👍

As I can't imagine it leading to regressions on other OS, I'll approve this.

@vince-fugnitto vince-fugnitto added the OS/Windows issues related to the Windows OS label Nov 2, 2021
@zhaomenghuan zhaomenghuan force-pushed the bugfix/electron-isDevMode branch from 41a260d to d83434b Compare November 13, 2021 15:26
…vMode API in Windows system in development mode

Signed-off-by: zhaomenghuan <zhaomenghuan02@meituan.com>
@msujew msujew requested a review from paul-marechal November 22, 2021 14:21
Copy link
Member

@paul-marechal paul-marechal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, thanks!

@paul-marechal paul-marechal merged commit 2ab4dcc into eclipse-theia:master Nov 22, 2021
@paul-marechal paul-marechal added this to the 1.20.0 milestone Nov 22, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
electron issues related to the electron target OS/Windows issues related to the Windows OS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants