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

Refuse to run on Windows 11 #869

Merged
merged 1 commit into from Jan 11, 2022
Merged

Conversation

edoardopirovano
Copy link
Contributor

Refuse to run on Windows 11 and Windows Server 2022 which are currently unsupported. Sample failing run (obtained by modifying one of our CI checks to run on windows-2022): https://github.com/github/codeql-action/runs/4779298247

Merge / deployment checklist

  • Confirm this change is backwards compatible with existing workflows.
  • Confirm the readme has been updated if necessary.
  • Confirm the changelog has been updated if necessary.

@edoardopirovano edoardopirovano requested a review from a team as a code owner January 11, 2022 17:57
src/util.ts Outdated Show resolved Hide resolved
@adityasharad
Copy link
Contributor

I just remembered the suggestion I made on the corresponding CLI PR: non-compiled languages are perfectly fine, and we should avoid blocking them from running on Windows 11. Could we move this check later and error only if there is a compiled language in use?

@edoardopirovano edoardopirovano force-pushed the edoardo/windows-11-error branch 3 times, most recently from 2c0998f to 8361428 Compare January 11, 2022 18:22
@edoardopirovano
Copy link
Contributor Author

Good point, done @adityasharad. Sample error on the latest commit: https://github.com/github/codeql-action/runs/4779658824

src/init-action.ts Outdated Show resolved Hide resolved
@edoardopirovano edoardopirovano merged commit faa9ba7 into main Jan 11, 2022
@edoardopirovano edoardopirovano deleted the edoardo/windows-11-error branch January 11, 2022 20:34
@github-actions github-actions bot mentioned this pull request Jan 11, 2022
5 tasks
edoardopirovano added a commit that referenced this pull request Feb 24, 2022
edoardopirovano added a commit that referenced this pull request Feb 24, 2022
Revert "Merge pull request #869 from github/edoardo/windows-11-error"
@github-actions github-actions bot mentioned this pull request Mar 7, 2022
5 tasks
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.

None yet

2 participants