Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Fix for #18462 and #18455 #18385

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

Closed
wants to merge 104 commits into from
Closed

Conversation

Harry-Hopkinson
Copy link

@Harry-Hopkinson Harry-Hopkinson commented Jan 27, 2022

Error message if you try to run code with no active text editor opened.
A vscode error message window will pop up with an error.
Fix for Error #18462 and #18455

@Harry-Hopkinson
Copy link
Author

@karrtikr are the tests good? - I have never wrote a test before.

@Harry-Hopkinson
Copy link
Author

@karrtikr I have changed the tests so this should be ready for review - I am justing importing from vscode for the IAppShell window parameter. If I have to change that, it is no problem.

@Harry-Hopkinson
Copy link
Author

@karrtikr I have also attempted to solve #18455 and #18462

@Harry-Hopkinson Harry-Hopkinson changed the title Error message if you try to run code with no active text editor opened. Error message if you try to run code with no active text editor opened. Fix for #18462 and #18455 Feb 7, 2022
@brettcannon
Copy link
Member

Please do not fix multiple issues in the same PR; it should be one PR per issue.

@brettcannon
Copy link
Member

I also just realized you did not follow https://github.com/microsoft/vscode-python/wiki/Submitting-a-pull-request#creating-a-pull-request and opened an issue first about what you're trying to change (which also says you should open one PR per issue).

Please read the development docs in our wiki before proceeding with this or any other PRs.

I have never wrote a test before.

I would advise reading up on software development to understand how to do testing, code organization/structure (e.g. when to write a class and when to write a function), etc. Our admittedly complicated code base may not be the best place to start with open source contributions if you still learning how to write tests.

@Harry-Hopkinson Harry-Hopkinson changed the title Error message if you try to run code with no active text editor opened. Fix for #18462 and #18455 Fix for #18462 and #18455 Feb 7, 2022
@Harry-Hopkinson Harry-Hopkinson changed the title Fix for #18462 and #18455 Fix for #18462 and #18455 and display an error message if you try to run code with no active text editor opened. Feb 7, 2022
@Harry-Hopkinson Harry-Hopkinson changed the title Fix for #18462 and #18455 and display an error message if you try to run code with no active text editor opened. Fix for #18462 and #18455 Feb 7, 2022
@brettcannon brettcannon removed their request for review February 8, 2022 00:52
@Harry-Hopkinson
Copy link
Author

I will close this as the problem has been solved

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

Successfully merging this pull request may close these issues.

3 participants