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

fix(autoclean): remove *.gz from the default .yarnclean file #4601

Merged
merged 1 commit into from
Oct 2, 2017

Conversation

nicobarray
Copy link
Contributor

Hello!

Summary

I removed the *.gz from the default .yarnclean generated by the yarn autoclean --init command.

Test plan

I tested by hand and launch yarn test and didn't find a test case for the autoclean --init option. If needed I could write a test case to check the default .yarnclean file content against the constant DEFAULT_FILTER.

I hope it resolve half the issue #4450 :)

Copy link
Member

@BYK BYK left a comment

Choose a reason for hiding this comment

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

Thanks!

@BYK BYK merged commit a3af477 into yarnpkg:master Oct 2, 2017
@BYK BYK changed the title ref(autoclean): remove *.gz from the default .yarnclean file fix(autoclean): remove *.gz from the default .yarnclean file Oct 2, 2017
@nicobarray nicobarray deleted the remove_gz_from_autoclean_init branch October 2, 2017 22:05
@nicobarray
Copy link
Contributor Author

;)

joaolucasl pushed a commit to joaolucasl/yarn that referenced this pull request Oct 27, 2017
…#4601)

**Summary**

I removed the *.gz from the default .yarnclean generated by the `yarn autoclean --init` command. 

**Test plan**

I tested by hand and launch `yarn test` and didn't find a test case for the autoclean --init option. If needed I could write a test case to check the default .yarnclean file content against the constant DEFAULT_FILTER.

I hope it resolve half the issue yarnpkg#4450 :)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants