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

Update system requirements for Sopel 7 #22

Closed
wants to merge 1 commit into from
Closed

Conversation

dgw
Copy link
Member

@dgw dgw commented Oct 5, 2019

We're ditching enchant and moving the rewritten spellcheck plugin (now using aspell) into its own project. Same for ipython; the eponymous plugin, though not rewritten, is no longer part of the base Sopel distribution.

If either sopel-irc/sopel#1675 or sopel-irc/sopel#1684 gets bumped to a later release (unlikely) or rejected (even less likely), this will need tweaking.

We're ditching `enchant` and moving the rewritten `spellcheck` plugin
(now using `aspell`) into its own project.

Same for `ipython`; the eponymous plugin, though not rewritten, is no
longer part of the base Sopel distribution.
@dgw dgw added the content label Oct 5, 2019
@dgw dgw added this to the Sopel 7.0.0 milestone Oct 5, 2019
dgw added a commit that referenced this pull request Jan 19, 2020
Not sure what I was doing back in October with that extra PR. Keeping
both of them would make merging a pain, so cherry-picking the extra
updates from there keeps the main "Sopel 7" branch nice and clean.
@dgw dgw removed this from the Sopel 7.0.0 milestone Jan 19, 2020
@dgw dgw added duplicate and removed content labels Jan 19, 2020
@dgw
Copy link
Member Author

dgw commented Jan 19, 2020

Pulled these changes into #17. There's no reason to have two separate PRs, and having both even makes the upcoming merge harder (they would conflict).

@dgw dgw closed this Jan 19, 2020
@dgw dgw deleted the update-reqs-7 branch January 19, 2020 08:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant