Akita 7 #1622
Closed
Akita 7 #1622
Conversation
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
tony
added a commit
that referenced
this pull request
Feb 5, 2022
tony
force-pushed
the
master
branch
5 times, most recently
from
48bf407
to
33c1e70
Compare
tony
force-pushed
the
master
branch
2 times, most recently
from
5777f9b
to
95a7f17
Compare
Even though we may not ultimately use it: - It seems to be a project in the midst of a philosophical problem. - Why give it another name, unless losing track of where it came from - It seems to decouple architecture (yay) but I'm concerned with not adopting an incremental approach to make yarn v1 better. Why not decouple yarn v1. Slowly improve that one. - 99% of the options berry has make no sense to me. Overwhelmed by choices. Every configuration option is listed, with no help given to making the bread and butter scenarios prominent. My guess is the settings are to fill niche caches and specialized project layouts. What would help: Show practical examples of what type of projects these configuration settings are for. If it's not fitting the scenario - don't even mention it. - Not a fan of how YN0000 shows everywhere. yarnpkg/berry#2517 since Feb 25, 2021 This may have a practical use case for it, e.g. when certain operations run in parallel. But this shows even when an operation isn't parallel. Eek. - The underlying feeling I have is berry is optimizing for the maintainers, and the user is an afterthought. This is direct opposite to what yarn is about. Yarn is about hiding complexity, user convenience and happiness, and performance. As odd as that may sound - it's important to strike a balance of a good product and a good developer experience. - I still see a lot of potential and benefit in berry As an open source maintainer The architecture is there, it just needs to get polished to show what matters to the user, to convey its value.
This reverts commit 6bc3542.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.