-
Notifications
You must be signed in to change notification settings - Fork 700
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
Add info on how to uninstall Dart #3187
Comments
Definitely a good idea, but may be hard to cover all possible forms of installation. With our current setup, the following cases seem the most important to cover initially:
It would also likely be good to mention removing Dart from path for situations where the package manager isn't responsible for doing so. We could consider a dedicated new page as well, for general tasks of uninstalling as well as cleaning up various PATH variables, tooling, etc. Also a friendly invitation back like you mentioned :) If anyone reading this wants to take one of those specific tasks, I'd be happy to help if you have any questions and work with you for review of a PR! Let me know :) |
I like the way you think, @parlough! |
Great points @parlough, |
There's also the removing Dart by removing Flutter case, right? |
This could just link back to the flutter site. flutter/website#5695 |
Absolutely.
…On Mon, May 3, 2021 at 9:40 AM Sameer Kashyap ***@***.***> wrote:
There's also the removing Dart by removing Flutter case, right?
This could just link back to the flutter site. flutter/website#5695
<flutter/website#5695>
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#3187 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AKS4PKJFEUOHYFYG6I7V3XTTL3GXDANCNFSM4345AKRA>
.
|
@parlough where would this page go? |
@Sameerkash I think a few places could use additions, with perhaps the most straightforward being how to uninstall with the tools mentioned in each tab on the Get the Dart SDK page. Those could be included near the end of each tab and perhaps completed as an initial PR separate from any new pages. As for the other pieces, such as removing extra items from the path, other forms of uninstall for manual installation, linking to the Flutter uninstall directions, etc I'm assuming we could create a new page as well. This could likely go in a similar place to the |
Like @parlough said, the uninstall page could be very similar to (and parallel to) the get-dart page. If the page starts looking too busy or you think that includes/tabs would be too hard to maintain, then we could use I don't think we need the actual uninstall directions on the get-dart page — just information on where to find the uninstall page. (I'm always leery of including instructions/code that you don't want everyone to copy-paste!) |
Okay, I'll try and make a draft PR soon, we can build things from there |
Fixes #3187 Fixes #5829 --------- Co-authored-by: Parker Lougheed <[email protected]>
Inspired by this issue on flutter.dev: flutter/website#5695
The text was updated successfully, but these errors were encountered: