Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make a whole bunch of small user experience improvements #60

Merged
merged 11 commits into from Jun 17, 2019

Conversation

Projects
None yet
2 participants
@nex3
Copy link
Contributor

commented Jun 4, 2019

No description provided.

nex3 added some commits Jun 4, 2019

Make MigratorRunner private
We don't want downstream Dart packages using this as an API, just as
an executable.
Mark some command-line flags as non-negatable
There's not much of a use-case for negating these, and this simplifies
the --help output.
Update the --verbose description
We want this description to be as general as possible, not specific to
--dry-run.

@nex3 nex3 requested a review from jathak Jun 4, 2019

@jathak

jathak approved these changes Jun 4, 2019

Show resolved Hide resolved README.md Outdated
Show resolved Hide resolved README.md Outdated

nex3 added some commits Jun 4, 2019

Update the README
It now refers to this package as a general migrator, includes build
status and download stats, and links to the (in-progress) Sass website
page that will include detailed installation and usage instructions.
Properly handle an un-passed --unicode flag
The args package never sets flags to null.

@nex3 nex3 force-pushed the ux branch from 8c8cdf9 to 2226e41 Jun 5, 2019

@nex3 nex3 merged commit 45c42db into master Jun 17, 2019

1 check passed

Travis CI - Pull Request Build Passed
Details

@nex3 nex3 deleted the ux branch Jun 17, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.