Skip to content
This repository was archived by the owner on Nov 1, 2024. It is now read-only.

add cli_logging.dart #15

Merged
merged 2 commits into from
May 17, 2017
Merged

add cli_logging.dart #15

merged 2 commits into from
May 17, 2017

Conversation

devoncarew
Copy link
Contributor

@pq

screen shot 2017-05-16 at 10 26 54 pm

screen shot 2017-05-16 at 10 27 37 pm

screen shot 2017-05-16 at 10 51 26 pm

pq
pq approved these changes May 17, 2017
View reviewed changes
Copy link
Contributor

@pq pq left a comment

Choose a reason for hiding this comment

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

Oh, you just new I'd love this. Awesome! 👍

Down the road, it'd be nice maybe to add some docs with screenshots but for now the example is great. Maybe update the README to point to it?

@devoncarew
Copy link
Contributor Author

sgtm re: doc - will update

@devoncarew
Copy link
Contributor Author

@pq, readme updated - can you take a pass through the docs? I think we'll want to publish a 0.1.0 shortly. Thanks!

@devoncarew devoncarew merged commit e62b70e into master May 17, 2017
@devoncarew devoncarew deleted the cli_logging branch May 17, 2017 17:00
@pq
Copy link
Contributor

pq commented May 17, 2017

Quick pass over docs. Look great!

mosuem pushed a commit to dart-lang/tools that referenced this pull request Oct 25, 2024
@devoncarew
Merge pull request dart-archive/cli_util#15 from dart-lang/cli_logging
5434b09 
add cli_logging.dart
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@pq pq pq approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
cla: yes
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

support some lightweight progress display
3 participants
@devoncarew @pq @googlebot