-
Notifications
You must be signed in to change notification settings - Fork 1.4k
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Recommend Yarn for installation (#1130)
- Loading branch information
1 parent
36ba8f2
commit 63e68fa
Showing
1 changed file
with
15 additions
and
1 deletion.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
63e68fa
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@sindresorhus this is objective and non-related to Ava, should be reverted and neutral, as tool decision should be left to the installer.
63e68fa
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PatrickHeneise We don't enforce anything, we're just recommending what we think is the best tool:
63e68fa
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The difference be it speed or performance or liking is better explained elsewhere, it has nothing to do with ava. I don't see the point in this readme.
63e68fa
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@PatrickHeneise AVA is heavy to install and using Yarn makes it much faster. Installing is the entry point to using AVA, and we want users to have a good experience there too. You might not see the point, but it's important for us. For reference: https://twitter.com/ava__js/status/785902327492333568 Let's end this discussion here. You're not going to convince us to change anything.
63e68fa
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well, neither are you convincing me to switch to yarn, so we're even here, I guess.