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

Configure CI testing on Windows #1330

Open
dgw opened this issue May 11, 2018 · 2 comments
Open

Configure CI testing on Windows #1330

dgw opened this issue May 11, 2018 · 2 comments

Comments

@dgw
Copy link
Member

dgw commented May 11, 2018

Travis is pretty great at what it does, but it only offers Unix-like environments (Linux & macOS). Since Appveyor is also available free for open-source projects, and runs its CI jobs on Windows, it seems like a no-brainer to run tests there as well.

At the moment, known problems on Windows include #1142 (pyenchant is not available for 64-bit Windows) and Appveyor builds are not necessarily expected to succeed. But with PRs like #1164 in progress, actual Windows support is not that far off. And since testing best-practice is to write a failing test first, then implement the functionality it checks, I'll be working on getting Appveyor configured properly for Sopel in the near future. (Status updates will be posted here as relevant.)

@dgw dgw self-assigned this May 11, 2018
@dgw dgw added this to the 7.0.0 milestone May 11, 2018
@dgw dgw modified the milestones: 7.0.0, 8.0.0 Oct 24, 2019
@dgw dgw changed the title Configure Appveyor for testing on Windows Configure CI testing on Windows Oct 24, 2019
@dgw
Copy link
Member Author

dgw commented Oct 24, 2019

Here's the updated plan for this:

  1. Defer to dev cycle for Sopel 8 (version 7 needs to release soon, and this would be a distraction)
  2. We'll probably use either Travis' new Windows environment or switch all CI to GitHub Actions, eliminating in either case the need to maintain more than one CI service config
  3. Reduced Python version support planned for Sopel 8 will make testing on additional OSes less ridiculous (imagine if we tested six supported Python branches on two or even three platforms…)

Note that the original impetus (known problems on Windows like pyenchant) won't matter any more—we're ditching enchant and removing the spellcheck plugin from core entirely. But we definitely should at least test on Windows with the latest Python version, just for verification's sake.

@dgw
Copy link
Member Author

dgw commented Jul 14, 2022

We still don't have much motivation to worry about this, given the existence of WSL. Dropping to wishlist.

@dgw dgw removed this from the 8.0.0 milestone Jul 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants