Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(29)

Issue 1153643002: Use "package:" imports for pub. (Closed)

Created:
5 years, 7 months ago by nweiz
Modified:
5 years, 7 months ago
Reviewers:
Bob Nystrom
CC:
reviews_dartlang.org
Base URL:
git@github.com:dart-lang/pub_test@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Patch Set 1 #

Total comments: 25

Patch Set 2 : Code review changes #

Unified diffs Side-by-side diffs Delta from patch set Stats (+239 lines, -220 lines) Patch
M bin/pub.dart View 1 chunk +1 line, -1 line 0 comments Download
M lib/src/dart.dart View 1 chunk +1 line, -2 lines 0 comments Download
M test/ascii_tree_test.dart View 1 chunk +2 lines, -3 lines 0 comments Download
M test/barback/fails_if_args_with_all_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/fails_if_dir_arg_does_not_exist_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/fails_if_dir_outside_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/fails_if_no_default_dirs_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/fails_on_all_with_no_buildable_directories_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/fails_on_disallowed_directories_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/fails_on_overlapping_directories_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/barback/utils.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/build/outputs_error_to_json_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/cache/add/bad_version_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/cache/add/missing_package_arg_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/cache/add/package_not_found_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/cache/add/unexpected_arguments_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/cache/repair/handles_failure_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/cache/repair/reinstalls_git_packages_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/dart2js/does_not_support_invalid_option_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/dart2js/reports_dart_parse_errors_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/dependency_computer/error_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/dependency_computer/utils.dart View 1 1 chunk +9 lines, -9 lines 0 comments Download
M test/descriptor.dart View 1 1 chunk +3 lines, -3 lines 0 comments Download
M test/descriptor/git.dart View 1 1 chunk +1 line, -2 lines 0 comments Download
M test/descriptor/tar.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/downgrade/dry_run_does_not_apply_changes_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/error_group_test.dart View 1 1 chunk +2 lines, -3 lines 0 comments Download
M test/get/git/check_out_unfetched_revision_of_cached_repo_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/git/dependency_name_match_pubspec_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/git/doesnt_fetch_if_nothing_changes_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/git/lock_version_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/git/locked_revision_without_repo_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/git/require_pubspec_name_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/hosted/get_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/get/hosted/gets_a_package_with_busted_dev_dependencies_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/get/hosted/stay_locked_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/hosted/unlock_if_version_doesnt_exist_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/path/empty_pubspec_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/path/nonexistent_dir_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/get/path/relative_path_test.dart View 1 1 chunk +2 lines, -2 lines 0 comments Download
M test/global/activate/activate_hosted_after_git_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/activate_hosted_after_path_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/activate_path_after_hosted_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/bad_version_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/constraint_with_path_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/empty_constraint_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/global/activate/missing_package_arg_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/path_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/unexpected_arguments_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/activate/unknown_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/binstubs/explicit_and_no_executables_options_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/binstubs/outdated_snapshot_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/binstubs/unknown_explicit_executable_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/deactivate/missing_package_arg_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/global/deactivate/path_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/deactivate/unexpected_arguments_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/global/deactivate/unknown_package_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/global/list_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/run/errors_if_outside_bin_test.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M test/global/run/implicit_executable_name_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/global/run/missing_executable_arg_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/global/run/missing_path_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/run/nonexistent_script_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/global/run/unknown_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/hosted/fail_gracefully_on_missing_package_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/hosted/fail_gracefully_on_url_resolve_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/implicit_barback_dependency_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/implicit_dependency_test.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M test/io_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/archives_and_uploads_a_package_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/cloud_storage_upload_doesnt_redirect_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/cloud_storage_upload_provides_an_error_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/does_not_publish_if_private_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/does_not_publish_if_private_with_server_arg_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/force_cannot_be_combined_with_dry_run_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/force_does_not_publish_if_private_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/force_does_not_publish_if_there_are_errors_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/force_publishes_if_tests_are_no_warnings_or_errors_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/force_publishes_if_there_are_warnings_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/many_files_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/package_validation_has_a_warning_and_continues_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/package_validation_has_a_warning_and_is_canceled_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/package_validation_has_an_error_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/preview_errors_if_private_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/preview_package_validation_has_a_warning_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/preview_package_validation_has_no_warnings_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/server_arg_does_not_override_private_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/lish/utils.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/list_package_dirs/ignores_updated_pubspec_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/list_package_dirs/includes_dev_dependencies_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/list_package_dirs/lists_dependency_directories_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/list_package_dirs/lockfile_error_test.dart View 1 1 chunk +2 lines, -2 lines 0 comments Download
M test/list_package_dirs/missing_pubspec_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/list_package_dirs/no_lockfile_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/list_package_dirs/pubspec_error_test.dart View 1 1 chunk +2 lines, -2 lines 0 comments Download
M test/lock_file_test.dart View 1 chunk +5 lines, -5 lines 0 comments Download
M test/oauth2/utils.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/oauth2/with_a_server_rejected_refresh_token_authenticates_again_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/package_list_files_test.dart View 1 1 chunk +3 lines, -3 lines 0 comments Download
M test/preprocess_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/pub_get_and_upgrade_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/pub_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/pub_uploader_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
M test/pubspec_test.dart View 1 chunk +5 lines, -5 lines 0 comments Download
M test/real_version_test.dart View 1 chunk +2 lines, -2 lines 0 comments Download
M test/run/errors_if_no_executable_is_given_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/run/errors_if_only_transitive_dependency_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/run/errors_if_path_in_dependency_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/run/nonexistent_dependency_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/run/nonexistent_script_in_dependency_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/run/nonexistent_script_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/serve/gets_first_if_dependency_is_not_installed_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/serve/gets_first_if_transitive_dependency_is_not_installed_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/serve/missing_file_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/serve/native_watch_removed_file_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/serve/utils.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M test/serve/watch_removed_file_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/serve/web_socket/path_to_urls_test.dart View 1 1 chunk +1 line, -1 line 0 comments Download
M test/serve_packages.dart View 1 chunk +2 lines, -2 lines 0 comments Download
M test/test_pub.dart View 2 chunks +19 lines, -19 lines 0 comments Download
M test/transcript_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/transformer/can_log_messages_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/transformer/fails_to_load_a_pubspec_with_reserved_transformer_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/transformer/fails_to_load_a_transform_from_a_deps_dev_dependency_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/transformer/fails_to_load_a_transform_from_a_non_dependency_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/upgrade/dry_run_does_not_apply_changes_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/upgrade/git/upgrade_to_incompatible_pubspec_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/utils_test.dart View 1 chunk +2 lines, -1 line 0 comments Download
M test/validator/compiled_dartdoc_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M test/validator/dependency_override_test.dart View 1 chunk +4 lines, -3 lines 0 comments Download
M test/validator/dependency_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M test/validator/directory_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M test/validator/executable_test.dart View 1 1 chunk +3 lines, -3 lines 0 comments Download
M test/validator/license_test.dart View 1 chunk +4 lines, -4 lines 0 comments Download
M test/validator/name_test.dart View 1 chunk +4 lines, -4 lines 0 comments Download
M test/validator/pubspec_field_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M test/validator/sdk_constraint_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M test/validator/size_test.dart View 1 chunk +1 line, -1 line 0 comments Download
M test/validator/utf8_readme_test.dart View 1 chunk +3 lines, -3 lines 0 comments Download
M test/version_solver_test.dart View 1 chunk +9 lines, -9 lines 0 comments Download

Messages

Total messages: 5 (0 generated)
nweiz
5 years, 7 months ago (2015-05-21 20:41:31 UTC) #1
Bob Nystrom
Lots of nits then LGTM! https://codereview.chromium.org/1153643002/diff/1/test/dependency_computer/utils.dart File test/dependency_computer/utils.dart (right): https://codereview.chromium.org/1153643002/diff/1/test/dependency_computer/utils.dart#newcode11 test/dependency_computer/utils.dart:11: Alphabetize these with the ...
5 years, 7 months ago (2015-05-22 00:05:17 UTC) #2
nweiz
Code review changes
5 years, 7 months ago (2015-05-22 00:11:06 UTC) #3
nweiz
Committed patchset #2 (id:20001) manually as a2373cc52677fb960781ed1c797fa5197db6cd8f (presubmit successful).
5 years, 7 months ago (2015-05-22 00:11:45 UTC) #4
nweiz
5 years, 7 months ago (2015-05-22 00:14:24 UTC) #5
Message was sent while issue was closed.
https://codereview.chromium.org/1153643002/diff/1/test/dependency_computer/ut...
File test/dependency_computer/utils.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/dependency_computer/ut...
test/dependency_computer/utils.dart:11: 
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Alphabetize these with the above imports.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/descriptor.dart
File test/descriptor.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/descriptor.dart#newcode12
test/descriptor.dart:12: 
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Ditto.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/descriptor/git.dart
File test/descriptor/git.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/descriptor/git.dart#ne...
test/descriptor/git.dart:9: import 'package:pub/src/git.dart' as git;
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> After path.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/error_group_test.dart
File test/error_group_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/error_group_test.dart#...
test/error_group_test.dart:12: import 'package:pub/src/utils.dart';
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Move up.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/get/path/relative_path...
File test/get/path/relative_path_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/get/path/relative_path...
test/get/path/relative_path_test.dart:10: import
'package:pub/src/source_registry.dart';
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> With the above.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/global/run/errors_if_o...
File test/global/run/errors_if_outside_bin_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/global/run/errors_if_o...
test/global/run/errors_if_outside_bin_test.dart:6: import
'package:path/path.dart' as p;
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Swap.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/implicit_dependency_te...
File test/implicit_dependency_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/implicit_dependency_te...
test/implicit_dependency_test.dart:9: import 'package:pub/src/barback.dart' as
barback;
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Move these above the relative ones.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/list_package_dirs/pubs...
File test/list_package_dirs/pubspec_error_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/list_package_dirs/pubs...
test/list_package_dirs/pubspec_error_test.dart:9: import
'package:pub/src/io.dart';
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Merge with above.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/package_list_files_tes...
File test/package_list_files_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/package_list_files_tes...
test/package_list_files_test.dart:12: import
'package:pub/src/system_cache.dart';
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Move.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/serve/utils.dart
File test/serve/utils.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/serve/utils.dart#newco...
test/serve/utils.dart:12: import 'package:http/http.dart' as http;
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Swap.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/serve/web_socket/path_...
File test/serve/web_socket/path_to_urls_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/serve/web_socket/path_...
test/serve/web_socket/path_to_urls_test.dart:8: import 'package:path/path.dart'
as p;
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Swap.

Done.

https://codereview.chromium.org/1153643002/diff/1/test/validator/executable_t...
File test/validator/executable_test.dart (right):

https://codereview.chromium.org/1153643002/diff/1/test/validator/executable_t...
test/validator/executable_test.dart:9: import
'package:pub/src/validator/executable.dart';
On 2015/05/22 00:05:17, Bob Nystrom wrote:
> Sort.

Done.

Powered by Google App Engine
This is Rietveld 408576698