Skip to content
This repository has been archived by the owner on Nov 6, 2019. It is now read-only.

Plugin refactor #191

Open
JulienMelissas opened this issue Dec 4, 2017 · 2 comments
Open

Plugin refactor #191

JulienMelissas opened this issue Dec 4, 2017 · 2 comments

Comments

@JulienMelissas
Copy link
Contributor

Hey all!

This weekend when working on the plugin I realized we've got a pretty major thing that could keep people from contributing or fixing existing issues easily: an 8000+ line plugin file.

I just wanted to put this here to get the conversation started! I'm thinking pulling the existing plugin code apart into a few major class files could really help.

@iandunn
Copy link
Contributor

iandunn commented Dec 6, 2017

👍

This isn't a huge priority, but would definitely be nice. Things like this tend to introduce unexpected bugs if they're not tested really well, though, so it'd probably be good to just start with just 1 PR for a medium-sized chunk, and get some others involved to test.

The bi-weekly bug scrubs in #meta-wordcamp on Slack would be a good place to recruit some people to help test.

@mcsmithers
Copy link

I'll try this one out. It will be a good opportunity to explore the codebase and help solve some bugs perhaps.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants