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

Merge with eslint-plugin-ember #73

Open
Turbo87 opened this issue Jul 7, 2017 · 7 comments
Open

Merge with eslint-plugin-ember #73

Turbo87 opened this issue Jul 7, 2017 · 7 comments

Comments

@Turbo87
Copy link

Turbo87 commented Jul 7, 2017

I don't know why this project was started when we also have https://github.com/netguru/eslint-plugin-ember but I think having two projects that try to do similar things is just wasting resources.

I would like to propose to merge this project with https://github.com/netguru/eslint-plugin-ember under a shared GitHub org so that everyone can act as a maintainer.

see ember-cli/eslint-plugin-ember#88

@rwjblue
Copy link
Collaborator

rwjblue commented Jul 7, 2017

This project started prior to the net guru project being renamed and popularized. I'm all for a shared collaboration in a repo with shared ownership. Happy to host that in this org (the org name seems to fit the purpose) if others are also 👍.

@scalvert
Copy link
Collaborator

scalvert commented Jul 7, 2017

I'm all for merging. I also think it makes sense to merge these rules into the netguru ones, and ultimately move that project under this org. We can all act as maintainers of the newly merged one.

@sandstrom
Copy link

Awesome to see that the two efforts are (or will be) combined! 🥇

What's the state of this merge?

It seems like the netguru repo has been moved to https://github.com/ember-cli/eslint-plugin-ember, but this one also seems to be active.

If eslint-plugin-ember has superseded this one, how about adding a large warning/note on the readme in this repo (I accidentally added this one to our project, before realising that a similar plugin existed under the ember-cli org).

@Turbo87
Copy link
Author

Turbo87 commented Sep 27, 2017

If eslint-plugin-ember has superseded this one, how about adding a large warning/note on the readme in this repo

I think that would be a good idea. Feel free to open a PR for this :)

@sandstrom
Copy link

sandstrom commented Sep 28, 2017

@Turbo87 Happy to submit a PR, just want to check with the repo-owner first.

@rwjblue Should I PR a readme change directing people to eslint-plugin-ember? (see my comment above)

@trentmwillis
Copy link
Member

I'm unsure about the other maintainers, but I never received any real notice of the merge occurring or being completed. If it has, then we likely have diverged again as there has been ongoing work in this repo.

@rwjblue
Copy link
Collaborator

rwjblue commented Sep 30, 2017

Agreed. We need to make a meta issue to evaluate and migrate or deprecate (with guidance) each individual rule.

A deprecation notice is a tad premature until we also provide guidance...

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

No branches or pull requests

5 participants