X Tutup
The Wayback Machine - https://web.archive.org/web/20220320094112/https://github.com/github/codeql-action/pull/869
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

Refuse to run on Windows 11 #869

merged 1 commit into from Jan 11, 2022

Conversation

edoardopirovano
Copy link
Contributor

@edoardopirovano edoardopirovano commented Jan 11, 2022

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 as a code owner Jan 11, 2022
src/util.ts Outdated Show resolved Hide resolved
@adityasharad
Copy link
Contributor

@adityasharad adityasharad commented Jan 11, 2022

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 Jan 11, 2022
@edoardopirovano
Copy link
Contributor Author

@edoardopirovano edoardopirovano commented Jan 11, 2022

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
214 of 348 checks passed
@edoardopirovano edoardopirovano deleted the edoardo/windows-11-error branch Jan 11, 2022
@github-actions github-actions bot mentioned this pull request Jan 11, 2022
5 tasks
edoardopirovano added a commit that referenced this issue Feb 24, 2022
This reverts commit faa9ba7, reversing
changes made to 71112ab.
edoardopirovano added a commit that referenced this issue 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
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
X Tutup