-
Notifications
You must be signed in to change notification settings - Fork 715
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
Revise Dart FAQ #5479
Revise Dart FAQ #5479
Conversation
Visit the preview URL for this PR (updated for commit 6ae11e6): |
Can you update the PR description to explain whether you checked the page for any info that might be beneficial to preserve since it doesn't exist anywhere else on the site. I can't go through all of them but at a glance some of these warrant at least a single sentence added to their relevant corresponding pages, e.g. Why are generics covariant? probably deserves a mention on the generics page, something about AOT vs JIT tradeoffs on the |
@MaryaBelanger : I did in part. Maybe removal isn't the right move here. I'll take another look. |
I think until someone has a chance to collect (today's) common questions and determine their answers, I think it's fine to delete this. It's one of the least visited pages. It's just someone needs to go through each answer to determine if it is still relevant and if so, should it be documented elsewhere? Some cases, such as in the browser version support change you already made, already should have been. Other answers, even if they're still relevant, might already be documented elsewhere or not important enough to document. I'm happy to discuss any of the entries if needed :) |
@parlough @MaryaBelanger : I did discuss with @munificent and his views matched @parlough . I moved some information, but didn't do all of it yet. @MaryaBelanger : Would you prefer to have a full accounting first or can I remove and we can revisit after I/O? |
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.
@atsansone please create the issue to revisit
Revised for currency. @MaryaBelanger : Created a draft issue to revisit later. Thanks! Fixes dart-lang#4673
Revised for currency. @MaryaBelanger : Created a draft issue to revisit later. Thanks! Fixes dart-lang#4673
Revised for currency. @MaryaBelanger : Created a draft issue to revisit later. Thanks! Fixes dart-lang#4673
Revised for currency.
Fixes #4673