-
Notifications
You must be signed in to change notification settings - Fork 59
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
Temporarily remove or fix Windows image in sbt-github-actions CI #123
Comments
The reason for Windows in CI is not for development, but for projects that are actually using this plugin on Windows. For example, Cats Effect includes Linux, macOS, and Windows in its CI matrix. If I recall correctly there are some subtle issues with line endings and such that need to be accounted for. |
So I am trying to solve this in #121 but as far as I can tell it doesn't have anything to do with line endings but rather just finding a specific file. |
If fixing the test on Windows is not easy, then I think it's fine to comment Windows CI out rather than blocking all other PRs. |
So I managed to fix the basic issue with the project not being able to load on Windows with #125 however I am now getting problems regarding the tests failing. These problems are different to the CI problems and seems be due to what @armanbilge was mentioning earlier, i.e.
I will make a separate issue for this specifically. |
So after fixing some line break problems in #127 and getting sbt-github-actions to pass all tests on my local Windows machine I think the core problem is something else. When you look at the failures at https://github.com/sbt/sbt-github-actions/actions/runs/3587764952/jobs/6038462004 you can see there are a lot of socket connection issues, i.e.
I think the problem here is that the Windows image in the CI is restricting the default port that is being used by the sbt server/client. @eed3si9n Maybe you have some idea here? |
So I found out from the sbt discord that this is not an isolated issue, it appears that scripted tests in general don't work in the github actions Windows CI, so I made a general issue on sbt at sbt/sbt#7082 |
Currently the CI is broken for Windows (see in progress PR at #121 trying to diagnose why). I have tried to setup this project on a Windows machine to help in finding out the problem but ended up failing to do so due to #122.
@eed3si9n @armanbilge Do you have any qualms in disabling this project on Windows for now? I don't think anyone is developing this plugin on Windows and its very easy to enable later on (just need to adjust https://github.com/sbt/sbt-github-actions/blob/main/build.sbt#L30).
The text was updated successfully, but these errors were encountered: