Skip to content

Create when offline style #101589

Merged
fluttergithubbot merged 17 commits intoflutter:masterfrom
Jasguerrero:create_when_offline
Apr 11, 2022
Merged

Create when offline style #101589
fluttergithubbot merged 17 commits intoflutter:masterfrom
Jasguerrero:create_when_offline

Conversation

@Jasguerrero
Copy link
Contributor

@Jasguerrero Jasguerrero commented Apr 8, 2022

Missing style nits from previous PR #100941

Pre-launch Checklist

  • I read the [Contributor Guide] and followed the process outlined there for submitting PRs.
  • I read the [Tree Hygiene] wiki page, which explains my responsibilities.
  • I read and followed the [Flutter Style Guide], including [Features we expect every widget to implement].
  • I signed the [CLA].
  • I listed at least one issue that this PR fixes in the description above.
  • I updated/added relevant documentation (doc comments with ///).
  • I added new tests to check the change I am making, or this PR is [test-exempt].
  • All existing and new tests are passing.

@flutter-dashboard flutter-dashboard bot added the tool Affects the "flutter" command-line tool. See also t: labels. label Apr 8, 2022
@Jasguerrero Jasguerrero changed the title Create when offline Create when offline style Apr 8, 2022
# Conflicts:
#	packages/flutter_tools/lib/src/runner/flutter_command.dart
#	packages/flutter_tools/test/commands.shard/hermetic/create_usage_test.dart
// First always update universal artifacts, as some of these (e.g.
// ios-deploy on macOS) are required to determine `requiredArtifacts`.
bool offline;
final bool offline;
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this can be a one-liner:

final bool offline = boolArg('offline') ?? false;

Copy link
Contributor

Choose a reason for hiding this comment

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

nvm, discussed offline

Copy link
Contributor

@christopherfujino christopherfujino left a comment

Choose a reason for hiding this comment

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

LGTM

@fluttergithubbot
Copy link
Contributor

This pull request is not suitable for automatic merging in its current state.

  • The status or check suite Google testing has failed. Please fix the issues identified (or deflake) before re-applying this label.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tool Affects the "flutter" command-line tool. See also t: labels.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants