Who uses Dart (updated) #4601
Draft
Who uses Dart (updated) #4601
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
workiva typo fix Co-authored-by: John Ryan <ryjohn@google.com>
workiva typo fix Co-authored-by: John Ryan <ryjohn@google.com>
Co-authored-by: MikiPaul <mikipaul46@gmail.com>
mit-mit
reviewed
Feb 21, 2023
src/community/who-uses-dart.md
Outdated
|
|
||
|
|
||
| Dart is used in almost every product group at Google: |
There was a problem hiding this comment.
Isn't this "by almost every product groups"? (product groups are people?)
src/community/who-uses-dart.md
Outdated
|
|
||
| * [BMW][] | ||
| > "The BMW Group is establishing itself as a leading automotive app developer |
There was a problem hiding this comment.
If we're keeping quotes, I think they should be ones that talk about Dart somehow.
And, yeah, the layout of these quotes looks pretty odd in a list like this...
Alternatively, we could just change this to short descriptions, and pull them up a level, e.g.:
* [BMW][]: A total of 300 developers use Dart to deliver a unified *My BMW app* for iOS and Android.
src/community/who-uses-dart.md
Outdated
| [Google One]: https://one.google.com/about | ||
| [Cloud Search]: https://workspace.google.com/intl/en_in/products/cloud-search | ||
|
|
||
| ## External |
There was a problem hiding this comment.
Not sure I understand how this is "External"? Should it be "Other"?
atsansone
requested changes
Feb 21, 2023
src/community/who-uses-dart.md
Outdated
Comment on lines
9
to
11
| The popular [Flutter toolkit][] is powered by the Dart language, | ||
| runtimes, tooling, and libraries. Over 500,000 Flutter apps for Android, | ||
| iOS, the web, and other targets, are all Dart-based. |
There was a problem hiding this comment.
issue: Active voice, per the GSG.
Suggested change
| The popular [Flutter toolkit][] is powered by the Dart language, | |
| runtimes, tooling, and libraries. Over 500,000 Flutter apps for Android, | |
| iOS, the web, and other targets, are all Dart-based. | |
| The Dart language and ecosystem powers the popular [Flutter toolkit][]. | |
| Developers have created more than 500,000 Flutter apps for Android, | |
| iOS, the web, and other targets using Dart. |
src/community/who-uses-dart.md
Outdated
|
|
||
|
|
||
| Dart is used in almost every product group at Google: |
There was a problem hiding this comment.
issue: Active voice.
Suggested change
| Dart is used in almost every product group at Google: | |
| Most product groups at Google use Dart: |
src/community/who-uses-dart.md
Outdated
Comment on lines
40
to
47
| * The [Google Play Console][] lets app publishers market Android apps, and | ||
| is written completely in Dart. | ||
| * The [Google Ads][] team continues to grow, with hundreds of SWEs primarily | ||
| writing Dart code. | ||
| * The [Google Assistant][] team uses Flutter and Dart to drive the overall system | ||
| UI of their Smart Displays. | ||
| * Both the UI and much of the back-end code for [Google Nest Hub][] and Hub Max | ||
| devices are written in Dart. |
There was a problem hiding this comment.
issue: Active voice.
Suggested change
| * The [Google Play Console][] lets app publishers market Android apps, and | |
| is written completely in Dart. | |
| * The [Google Ads][] team continues to grow, with hundreds of SWEs primarily | |
| writing Dart code. | |
| * The [Google Assistant][] team uses Flutter and Dart to drive the overall system | |
| UI of their Smart Displays. | |
| * Both the UI and much of the back-end code for [Google Nest Hub][] and Hub Max | |
| devices are written in Dart. | |
| * The Google Play team writes the [Google Play Console][] in Dart. | |
| It lets app publishers market Android apps. | |
| * The [Google Ads][] team continues to grow with hundreds of | |
| SWEs writing Dart code. | |
| * The [Google Assistant][] team uses Dart and Flutter to drive the | |
| user interface of their Smart Displays. | |
| * The Google Nest team wrote the user interface and much of the backend code | |
| for [Google Nest Hub][] and Hub Max devices in Dart. |
src/community/who-uses-dart.md
Outdated
Comment on lines
59
to
60
| Several other Google teams are currently rewritting their projects in Flutter. | ||
| In all, Google has thousands of developers writing millions of lines of code in Dart. |
There was a problem hiding this comment.
issue: Active voice. Also, spelling.
Suggested change
| Several other Google teams are currently rewritting their projects in Flutter. | |
| In all, Google has thousands of developers writing millions of lines of code in Dart. | |
| The number of Google teams rewriting their projects in | |
| Dart and Flutter continues to grow. | |
| In all, Google has thousands of developers writing millions | |
| of lines of code in Dart. |
src/community/who-uses-dart.md
Outdated
Comment on lines
76
to
82
| Beyond Google and Flutter, many companies use Dart for building their applications: | ||
|
|
||
| * [Workiva's][] next-generation client applications for their productivity suite | ||
| are built with Dart. | ||
| * [Wrike][] uses the [AngularDart][] framework to develop project management tools. | ||
| * [Blossom][] is an agile project management tool built with Dart. | ||
| * And many more! |
There was a problem hiding this comment.
question: Should these not be combined with the quoted ones listed previously? If so, then move then and rewrite with active voice.
src/community/who-uses-dart.md
Outdated
| * And many more! | ||
|
|
||
| If you build production software with Dart and want to be included in this list, | ||
| please visit [our site's GitHub] to open an issue or pull request, thanks! |
There was a problem hiding this comment.
src/community/who-uses-dart.md
Outdated
| [Wrike]: https://www.wrike.com | ||
| [AngularDart]: https://angulardart.xyz/guide/setup | ||
| [Blossom]: https://blossom.co | ||
| [our site's GitHub]: https://github.com/dart-lang/site-www |
There was a problem hiding this comment.
issue: To match the prior change.
Suggested change
| [our site's GitHub]: https://github.com/dart-lang/site-www | |
| [Dart.dev GitHub repo]: https://github.com/dart-lang/site-www |
atsansone
added
review.await-update
|
Hey @MaryaBelanger would be great to get this one landed now that we're past Dart 3. |
|
@mit-mit sure thing, getting back to it now. |
|
Visit the preview URL for this PR (updated for commit 3179e76): |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a revision of #4577 (closing that in favor of this, added original author as co-author here) because I couldn't add changes to that contributor's fork. Thanks for getting this started @MikiPAUL!
Fixes #4144
Question for reviewers: I'm unsure why the page was taken down originally (archived here). Should I add that content to this PR?