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

docs: fix typo in --defaults description #1540

Closed
wants to merge 2 commits into from

Conversation

snitin315
Copy link
Member

@snitin315 snitin315 commented May 13, 2020

What kind of change does this PR introduce?

typo fix

Did you add tests for your changes?
NA
If relevant, did you update the documentation?
YES
Summary
aggresively aggressively

Does this PR introduce a breaking change?
NO

Other information
NA

Copy link
Member

@jamesgeorge007 jamesgeorge007 left a comment

Choose a reason for hiding this comment

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

Please update all instances

@webpack-bot
Copy link

@snitin315 Thanks for your update.

I labeled the Pull Request so reviewers will review it again.

@jamesgeorge007 Please review the new changes.

@snitin315
Copy link
Member Author

I think we should add a snapshot test for --help. What do you say?

@snitin315 snitin315 changed the title chore: fix typo in --defaults description docs: fix typo in --defaults description May 13, 2020
@snitin315 snitin315 closed this May 13, 2020
@snitin315 snitin315 reopened this May 13, 2020
@jamesgeorge007
Copy link
Member

I think we should add a snapshot test for --help. What do you say?

I think what we have now for help would suffice.

@anshumanv
Copy link
Member

We'll be removing this flag as per discussion in #1323
Closing

@anshumanv anshumanv closed this May 13, 2020
@snitin315
Copy link
Member Author

We'll be removing this flag as per discussion in #1323
Closing

I'm happy to send a PR for that as well, if you agree.

@anshumanv
Copy link
Member

WIP

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

Successfully merging this pull request may close these issues.

4 participants