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 lsp-ci.yaml [DO NOT MERGE] #772

Closed
wants to merge 9 commits into from
Closed

Update lsp-ci.yaml [DO NOT MERGE] #772

wants to merge 9 commits into from

Conversation

dogusata
Copy link
Contributor

@dogusata dogusata commented Feb 5, 2025

As github replaces the windows server version, even though they indicate that windows-latest = windows-2022, our tests started to fail in the same time for windows. To check if it is the case, manually setting the windows version on github action to windows-2022

Problem

Windows tests are failing on github actions

Solution

Instead of using windows-lates manually setting windows-2022 to check if that is the issue.

License

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

As github replaces the [windows server version](https://github.blog/changelog/2025-01-30-larger-hosted-runner-enhancements-edit-runner-size-and-windows-server-4vcpu-runner-availability/), even though they indicate that [windows-latest = windows-2022](https://github.com/actions/runner-images?tab=readme-ov-file#available-images), our tests started to fail in the same time for windows. To check if it is the case, manually setting the windows version on github action to windows-2022
@dogusata dogusata requested a review from a team as a code owner February 5, 2025 10:59
@dogusata dogusata changed the title Update lsp-ci.yaml Update lsp-ci.yaml [DO NOT MERGE] Feb 5, 2025
@dogusata dogusata closed this Feb 5, 2025
@dogusata dogusata deleted the dogusata-patch-1 branch February 5, 2025 17:14
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

Successfully merging this pull request may close these issues.

1 participant