Added info about the vscode editor #540
Conversation
|
Hi,
I'm very wary of recommending this as a default choice. |
|
May doesn't mean does, but it does and you have to opt-out. https://code.visualstudio.com/docs/supporting/faq#_how-to-disable-telemetry-reporting Does Atom have opt-in for this? |
|
Important note here - this is used for beginners PyLadies courses. Changing the default cannot happen without consulting the coaches. |
|
Ok, I changed things you said. As @benabraham said it can be diabled. @hroncok How I can contact them or consult it? |
|
I'll lure them here. |
|
Oh, okay thanks! |
|
Personally, I'm alright with it, use it all the time, and it is faster than Atom in my experience (by which I mean timings, so, time to start up, etc.). Considering privacy: VSCodium comes to the rescue here, with VSCode builds without the Microsoft stuff - not a recommendation, since I haven't actually tried out VSCodium myself. Then again, you can also just disable telemetry in VSCode. |
|
@golyalpha So is everything ok? Can this PR be merged? I see some error in travis |
I'd argue that for beginners we don't want those things. We need to explain how they actually work and they should not be inclined to "click a magic button" instead. |
|
Yes, there was "run from terminal" text. But someone here said that terminal will be explained in next lessons. So instead of that there is this "magic button". But I think there is no reason explain anything in "python , file name". And it will be faster to test and deploy code. |
|
@hroncok Read the next line. |
|
I'd argue against any button that runs Python code. Note that it's not productivity that we are trying to achieve, but understanding. |
|
@hroncok there are two options. Install the run code extension or run code via terminal. |
|
@hroncok I'm not arguing that. I also mentioned, that VSCode doesn't offer that kind of functionality by default, so I don't think that this argument is even appropriate for this kind of discussion. Tell me if I'm wrong though. |
|
I think that beginner will not understand theese things. VSCode is powerful code editor. And thats all. Beginners dont know theese things. |
|
Honestly, if we're about to get into a discussion about what VSCode with extensions is capable of, and how we don't want that, we should also remember that Atom has addons that half the time do much of the same things. |
|
I don't know either of those editors. So I'll crawl back under my rock. Sorry about misdirecting this discussion. |
|
Yet note that the text currently actually talks about some kind of extension installation. |
|
@golyalpha yes, yes... I just added info about good text editor that beginners will have that option to choose what is best for them. And thats all. @hroncok no problem. Dont worry about that. |
|
Ok I changed vscode to visual studio code and deleted the plugin tutorial. And fixed some typos |
Remove the "aby zlepšil uživatelské prostředí" (as of now, the Privacy Policy lists 17 reasons data is collected!). Include a note that the telemetry may send the content of your files. Citing the current privacy policy: > The data we collect can include the following: > [...] > **Text, inking, and typing data.** Text, inking, and typing > data and related information. > [...] > **Content.** Content of your files and communications you > input, upload, receive, create, and control. That might cover anything you type in or open in the editor. (Since the telemetry is closed-source, it's hard to tell if they're actually sending this.)
This is written for people who don't know Atom, so that reference needs to go. Also, we don't need to tell them how easy the installation is. They'll try, and if they find it easy, they'll know!
|
I'd be OK merging this with these changes: https://github.com/TomasRoj/naucse.python.cz/pull/1
|
|
I don't fully understand what do you mean with move atom to notepad ++. |
Fix up VS Code instructions
|
Ok merged that |
|
So is everything ok from my side? |
|
So can we merge? |
|
Tomáš, please, stop doing this. I would consider it OK to send bumps after this is ignored for couple days, but repeatedly asking "can we merge this? can we merge this?" after every comment is very annoying and may be considered as rude. Please, show some patience. This has high changes for being merged as is, yet there is still one detailed open question about subpages ordering. Ultimately, I leave it to @encukou to decide how and when to merge this. |
|
Sorry |
|
Merged! Thanks for initiating the change and all the discussions. |
|
Thank you so much, I am glad that I can helped. |
Hey there,
I added some info about he vscode editor from microsoft. I added because many people are switching to vscode right now and beginners should have option to choose. I dont know if my text is correctly modified so I am open to any tip/requests for code change.