Skip to content

Add Flutter 2.0.1 CLI Autocompletion - #40

Merged
rolandtshen merged 11 commits into
withfig:masterfrom
ampersanda:feat/flutter-2.0.1-autocompletion
Mar 25, 2021
Merged

Add Flutter 2.0.1 CLI Autocompletion#40
rolandtshen merged 11 commits into
withfig:masterfrom
ampersanda:feat/flutter-2.0.1-autocompletion

Conversation

@ampersanda

Copy link
Copy Markdown

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
Feature

What is the current behavior? (You can also link to an open issue here)
Flutter 2.0.1 command autocompletion is not implemented yet

What is the new behavior (if this is a feature change)?
Add Flutter 2.0.1 command autocompletion

Additional info:
None

Comment thread specs/flutter.js Outdated
Comment on lines +1243 to +1246
help,
verbose,
deviceUser,
deviceTimeout,

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks like the docs say there's a -d option that takes a device id as an argument

@ampersanda ampersanda Mar 20, 2021

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh yeah, I missed that, I should better use globalOpts instead because I saw globalOpts could be used everywhere

Comment thread specs/flutter.js Outdated
Comment on lines +749 to +779
name: "aar",
description: "Build a repository containing an AAR and a POM file."
},
{
name: "apk",
description: "Build an Android APK file from your app.",
},
{
name: "appbundle",
description: "Build an Android App Bundle file from your app."
},
{
name: "bundle",
description: "Build the Flutter assets directory from your app."
},
{
name: "ios",
description: "Build an iOS application bundle (Mac OS X host only)."
},
{
name: "ios-framework",
description: "Produces .xcframeworks for a Flutter project and its plugins for integration into existing, plain Xcode projects."
},
{
name: "ipa",
description: "Build an iOS archive bundle (Mac OS X host only)."
},
{
name: "web",
description: "Build a web application bundle."
},

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These seem to be suggestions that would go under a suggestions array in the args object, instead of options

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah sure, I replaced this to suggestions instead of options in the latest commit

Comment thread specs/flutter.js Outdated
{
name: ["-t", "--template"],
insertValue: "--template {cursor}",
description: "Specify the type of project to create. [app] (default) Generate a Flutter application. [module] Generate a project to add a Flutter module to an existing Android or iOS application. [package] Generate a shareable Flutter project containing modular Dart code. [plugin] Generate a shareable Flutter project containing an API in Dart code with a platform-specific implementation for Android, for iOS code, or for both.",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Large spacing

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Removed in latest commit

Comment thread specs/flutter.js Outdated
{
name: ["-r", "--reporter"],
insertValue: "--reporter",
description: "Set how to print test results.\n[compact] (default) A single line, updated continuously.\n[expanded] A separate line for each update.\n[json] A machine-readable format (see https://dart.dev/go/test-docs/json_reporter.md).\n(defaults to \"compact\")",

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

large spacing

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for pointing out. Removed in latest commit 👏

@rolandtshen

Copy link
Copy Markdown
Contributor

Thanks for making the changes! This was a big PR so thanks for the awesome contribution

@rolandtshen
rolandtshen merged commit 6d52f10 into withfig:master Mar 25, 2021
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.

2 participants