enzymejs / enzyme

JavaScript Testing utilities for React
https://enzymejs.github.io/enzyme/
MIT License
19.96k stars 2.01k forks source link

Projectname #43

Closed hkjels closed 8 years ago

hkjels commented 8 years ago

This can be confused with the ClojureScript view-layer over React called Reagent. It should be mentioned in the Readme.

ljharb commented 8 years ago

The tagline is "JavaScript Testing utilities for React" - is that not sufficient?

Is there a corresponding issue filed on the ClojureScript project to mention this project in the readme?

hkjels commented 8 years ago

It might, I am not the author, but if I was I would have liked a mention. Note that Reagent was developed before this project.

Kauko commented 8 years ago

I think it would be worthwhile coming up with a different name. Reagent is very established in the Clojure(Script) community, and there's a lot of overlap between the JS and CLJS communities.

Not sure exactly how old the clojurescript Reagent project is, but there's commits from at least 2013.

mike-thompson-day8 commented 8 years ago

For many people, Javascript and Clojurescript are in the same space. Just Google "React Reagent".

Look, we know it is a pain to rename, but really you should. There's just too much overlap here. And this project is about two years too late to the name.

alexkalderimis commented 8 years ago

This is particularly important since both projects are in the React ecosystem (hence the name choices obviously). Confusion is inevitable. https://github.com/reagent-project/reagent clearly has priority.

danielcompton commented 8 years ago

Think about this if the shoe was on the other foot. Imagine this library was open source for two years and then Google released an open source project focused on React also called Reagent. You'd probably feel pretty ticked off (I know I would).

Gregg8 commented 8 years ago

+1 for renaming. This is definitely going to cause confusion, being in such a similar space.

This Reagent's first commit was Aug 2015. The original Reagent has been going since December 2013 and is very well established.

brendonrapp commented 8 years ago

Both of these libraries relate to the React ecosystem. There's no reason this library should use a name that's already taken by something else within the same general sphere. It's not a good thing for anybody.

skovsgaard commented 8 years ago

Renaming would avoid the awkward PR situation Groupon ended in when they stole a GNOME project name and refused to acknowledge the conflict; it'd just be so much easier to change the name.

hipitihop commented 8 years ago

+1 for rename. You gain nothing from re-using the name in a related space 2 years after the fact, while both the existing and future communities for both projects will lose with confusion while looking up references

vibl commented 8 years ago

If you keep this name:

In FOSS communities, we don't privatize words by putting a trademark on them and suing anybody that uses it for something else in the same space (like you would for 'Airbnb'). We count on people's good judgement to respect the name of other libraries and sub-communities within the community.

sharnik commented 8 years ago

Rename, please. Not cool.

julianjelfs commented 8 years ago

You should absolutely rename and it's surprising that you made such a mistake in the first place.

martinklepsch commented 8 years ago

@julianjelfs please watch your tone. While this is unfortunate there is no need to attack people. Thank you :+1:

rwaguy commented 8 years ago

It would be a good idea to rename this project. Using the same name will confuse Clojure, Airbnb, and React users. While Airbnb may not feel this is a significant issue, you may want to consider that the programmers and companies who depend upon Clojure/Reagent may be among your clientele and investors, as well.

A partial list of those is here: http://clojure.org/Companies

domgetter commented 8 years ago

@martinklepsch Claiming that someone made a mistake is not an "attack". There is no need for you to claim false things here. Thank you.

Kauko commented 8 years ago

@domgetter the message he's replying to had a different tone before it was edited. It wasn't terrible before, but it's definitely more constructive now.

DavidYKay commented 8 years ago

Big user of the other Reagent, so I am a bit biased, but I think it'd be best for both the Clojurescript and Javascript programmers to keep the names separate. Please consider renaming. Thanks!

fromheten commented 8 years ago

:+1: for renaming. Great stuff otherwise :)

jstewart commented 8 years ago

There's already one very popular react library named "reagent". What the host language is should have no bearing as both use React, and this WILL cause confusion if you don't.

cjohansen commented 8 years ago

Seriously, can't you even be bothered to google a name before choosing it? It would've taken all of 10 seconds to figure out this name is already being used for something very similar (a React-related OSS project).

escherize commented 8 years ago

:+1: On the renaming. I was excited, thinking this was for testing reagent views...

skrat commented 8 years ago

You, you copycats! Much lame, MUST rename

jcoyne commented 8 years ago

Please pick a unique name that makes it easy to google your docs without getting hits for different products that are very similar.

vibl commented 8 years ago

I think a lot of people would be willing to help you with the renaming: if you announce another name and you ask for help, PRs with the new name replaced everywhere in the code and documentation would probably save you most of the grunt work.

Granted, the main hassles of a renaming generally is in the legacy code and docs, in changing internal habits and in public relations. But it can be done! (and keeping the name would create even more hassles in the medium and long terms).

maacl commented 8 years ago
ljharb commented 8 years ago

While we appreciate everyone offering input, and will consider it, please do not offer simple "+1s", as these add no value. I will be deleting all +1s that offer nothing to the conversation, and if more continue to be added, I will be forced to lock the conversation.

rocky-jaiswal commented 8 years ago

Please rename. There is a popular ClojureScript library with the same name. This will do nothing except confuse people.

gadfly361 commented 8 years ago

@ljharb I greatly appreciate that the rename will be considered! :)

I would submit, however, that the volume of +1s is significant and does affect the conversation - namely that this is important to an overwhelming number of people. Deleting the +1s is unnecessary, given that they are easy to skip past when skimming the thread.

csummers commented 8 years ago

Please rename. It creates confusion when googling.

Also, the value of a +1 is an indicator of the magnitude of the issue.

tiye commented 8 years ago

Maybe like "polyglot.js", just call it "reagent.js", will that help?

ljharb commented 8 years ago

@jiyinyiyong a ".js" suffix is a bit dated, so i wouldn't be in favor of using that convention for projects moving forward. @gadfly361 i don't find them easy to skip past when I'm trying to read the whole thread to understand people's thoughts. +@csummers: Issues (of any kind) are not decided based on "popularity" or "number of people affected" - 1 person is just as important as 1 million. The only impact "+1" has is irritating contributors; it doesn't increase the perceived importance of the issue.

julianjelfs commented 8 years ago

@ljharb Of course the +1s add value. They show clearly the level of feeling. If you delete them it just looks like you are in denial.

burhanloey commented 8 years ago

Please consider renaming this project. Both are used in JavaScript environment AND for React, which is not cool.

vibl commented 8 years ago

The +1s are the most important information in this thread.

This is a community problem. The community should, if not decide, at least be heard.

There aren't any representatives here, so the only voices are members voices.

The number of people that are affected by the name collision and agree about the best way to solve the problem is the one thing that should be considered in priority.

I hope as much people as possible will express their interest, by +1, -1 or a comment.

ioRekz commented 8 years ago

I could actually use both library in my project so I would like to see a way to differentiate them when googling reagent.

chepprey commented 8 years ago

I agree that the Clojurescript Reagent project rightfully has dibs on the name, and the fact that both of these projects are related to react.js would cause undue confusion.

How about we come up with some cool react-ish alternative names for this Airbnb project? How about "Reaction"? "Reactant"? "Reactest"?

ljharb commented 8 years ago

@vibl as I've said, the number of people is irrelevant, because one person carries as much weight to us as one million. No voices are needed because the issue has been raised, and we are considering it. Additional points - that have not already been raised - are indeed valuable, and I hope people will continue to add those.

So far, what's been raised is:

Please rest assured that even if no further comments were added to this thread, we will be considering it with the same amount of speed and attention than we would if thousands of people cluttered our email inboxes with "+1" notifications :-) (perhaps faster without the clutter!)

Alternate name suggestions are indeed welcome as well, but it might also help to put those in a separate issue.

mtrimpe commented 8 years ago

Definitely agree that the project should be renamed.

Reagent is a very popular key component for the (rather big) ClojureScript React community. This will definitely cause a lot of confusion and do a lot of damage to their project (and this one too.)

By itself that's hopefully plenty motivation to find an even better name; but also keep in mind that trying to keep the name would be very bad PR for AirBnB in the open source community.

Hope you choose to do the right thing! The project looks pretty cool...

milt commented 8 years ago

:+1: Please rename. Trust me, you don't want that name due to the chemistry connection, my search history looks like I've been trying to stock up a meth lab.

postspectacular commented 8 years ago

+1 for renaming! Reagent is established for 2+ years and one of the most prominent CLJS projects. I'd hate to see you just steamrolling it into invisibility due to your project's larger mass appeal to the pure-JS crowd. There're enough other names to choose from!

lelandrichardson commented 8 years ago

Thank you to everyone in this thread for your thoughts. A project rename is being considered.

I would like to add to @ljharb's comments the following:

  1. Please no more "+1" comments in this thread. We see that there are people who feel this needs to happen, and additional "+1" comments will no longer affect our decision. If you have more to say then please say it, however.
  2. To add a bit of context, the project was just renamed from catalyst internally. To be fair, this could have been done with a bit more research, but the main measure of "name is available" we were looking for was the package name being available on npm. Lesson learned.
  3. If this project is to be renamed, I would like for it to happen FAST.

Airbnb (and myself) of course want to be good stewards to this project and to the open source community at large, and have no desire to do it damage.

If you have any name recommendations, please put them in https://github.com/airbnb/reagent/issues/46

dviramontes commented 8 years ago

I second @postspectacular Please consider renaming. I use ClojureScript/Reagent every day and it's a formidable project. And thank you for making a react testing library, I also think that's a commendable effort!

jbeja commented 8 years ago

Thank you for reconsider.

maacl commented 8 years ago

@lelandrichardson @ljharb Ref your comments on +1 comments you guys have a lot to learn about running an open source project, if you consider the minor inconvenience it may cause you more important that letting the community voice its opinion.

mkcp commented 8 years ago

Maintainers: thank you for your consideration. As a user of cljs Reagent on a daily basis, I'm concerned that the name clash will be unhealthy for both projects in the long run.

I imagine the worst side effect would be on cljs devs when attempting to use this project in a ClojureScript environment.

I urge the maintainers to take us into consideration and use this time to pick a new name for http://github.com/airbnb/reagent.

hzhu commented 8 years ago

I ask that Airbnb rename this open source project.

Big fan of Airbnb and Airbnb engineering. I attend Airbnb Nerds meetup at your HQ as often as I can. We use Reagent in production at the company I work at. Though I use Reagent at work (and sometimes side projects), I consider myself first and foremost a JavaScript developer. The Reagent (CLJS) community really considers themselves not a separate community (isolated from React/JavaScript) both part of both, a joined community.

It would really suck to see Airbnb, not rename, and steamroll the existing community & open source project. Ya'll got nearly 500 stars in a few days.

Also the +1 comments though minimal in value, showed how many people we're affected or care about this issue. So far 44 of those +1 comments have been silenced and removed.

ljharb commented 8 years ago

@hzhu I don't think playing the game of "more people means it matters more" is going to be productive, especially if people start comparing the sizes of the JS and ClojureScript communities, or the number of stars a project has, etc. It's sufficient to say that any nonzero number of people are impacted, and that absolutely is enough to warrant consideration, with any issue, at any time, in any context.

@maacl thanks for your input. I'm following the lead of many other popular open source projects, including npm itself, in not conflating "the community voicing its opinion" with "let's all dogpile on a thread without adding any value". I appreciate that everyone wants to be heard, but that doesn't automatically guarantee that what they have to say is useful to anybody.

Thanks again to everybody for their input. Please follow along on #46 for naming suggestions, and please try to refrain from commenting unless you have a new thing for us to consider that adds to this list.

ericqweinstein commented 8 years ago

^ Apologies, I missed @lelandrichardson's comment above. My thought process:

bhugueney commented 8 years ago

Thanks for considering renaming the project. Names should be "locally" unique to avoid confusion. Javascript React is a specific enough space so that names uniqueness is a realistic goal. It will be a one time pain for you to switch, but taking the high road will be repaid by the goodwill you will reap from it.