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

Remove aggregator module from core #199

Closed
jenlampton opened this issue Mar 16, 2014 · 3 comments
Closed

Remove aggregator module from core #199

jenlampton opened this issue Mar 16, 2014 · 3 comments

Comments

@jenlampton
Copy link
Member

I'd like to propose that we remove aggregator module from core. I suspect that is not used on 80% of all Drupal sites. To date, I have only used it a handful of times, most of those it was eventually replaced with feeds (or FeedAPI back in D6) anyway.

I'd love to hear what others think about this proposal. Is aggregator module worth keeping around?

See also: #7

@DirectorHaas
Copy link

Agreed to removal. Yes, Feeds is preferred in any reasonably complex case. Small is beautiful.

@quicksketch
Copy link
Member

Back when we were trimming core in #7, I advocated for keeping Aggregator. However, after months of maintenance on it and the amount of enthusiasm keeping it is low, I agree we should remove it as well. Like other modules, I'd like to see this "contrib-ized", but for now taking it out so we don't have to manage it sounds like a good move to me.

@jenlampton filed a PR at backdrop/backdrop#184, which I've merged in. Thanks!

@oadaeh
Copy link

oadaeh commented Apr 19, 2014

I think it would be a good thing to reference documentation on how to "contrib-ize" a core module when removing it. That would make it easier for someone to pick up the gauntlet on a task like this.

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

4 participants