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

Remove win32-ia32 target #195559

Merged
merged 5 commits into from Oct 16, 2023
Merged

Remove win32-ia32 target #195559

merged 5 commits into from Oct 16, 2023

Conversation

joaomoreno
Copy link
Member

@joaomoreno joaomoreno commented Oct 13, 2023

This not only removes Windows 32-bit as a VS Code target, but ia32 support altogether since win32-ia32 was the last ia32 target supported. The last 32-bit target supported is linux-armhf.

To do before merging:

  • Test CI & build pipelines
  • Merge distro and update distro commit

Would be great to get the following eyes on this:

@joaomoreno joaomoreno self-assigned this Oct 13, 2023
@joaomoreno joaomoreno added this to the October 2023 milestone Oct 13, 2023
@joaomoreno joaomoreno added the engineering VS Code - Build / issue tracking / etc. label Oct 13, 2023
@joaomoreno joaomoreno marked this pull request as ready for review October 13, 2023 14:29
connor4312
connor4312 previously approved these changes Oct 13, 2023
Copy link
Member

@connor4312 connor4312 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Long live 64 bit 🔥 🔥

roblourens
roblourens previously approved these changes Oct 13, 2023
Copy link
Member

@roblourens roblourens left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Gonna tell the grandkids that I worked on 32-bit software

@joaomoreno joaomoreno dismissed stale reviews from roblourens and connor4312 via d5aa58e October 16, 2023 09:29
@joaomoreno joaomoreno enabled auto-merge (squash) October 16, 2023 09:30
@sandy081
Copy link
Member

@joaomoreno Should we also drop the support for win32-ia32 target for platform specific extensions also?

@joaomoreno
Copy link
Member Author

@joaomoreno Should we also drop the support for win32-ia32 target for platform specific extensions also?

Yes, this will follow suit.

sandy081
sandy081 previously approved these changes Oct 16, 2023
@joaomoreno joaomoreno merged commit 9905225 into main Oct 16, 2023
6 checks passed
@joaomoreno joaomoreno deleted the joao/remove-win32-ia32 branch October 16, 2023 11:28
@alexr00 alexr00 restored the joao/remove-win32-ia32 branch October 16, 2023 14:56
@alexr00 alexr00 deleted the joao/remove-win32-ia32 branch October 16, 2023 14:56
Alex0007 pushed a commit to Alex0007/vscode that referenced this pull request Oct 26, 2023
* remove win32-ia32

* update distro
nvarner added a commit to nvarner/typst-lsp that referenced this pull request Nov 1, 2023
Based on microsoft/vscode-vsce#908 and microsoft/vscode#195559, it
appears that extensions should no longer be published for the now
removed win32-ia32 target.
nvarner added a commit to nvarner/typst-lsp that referenced this pull request Nov 1, 2023
Based on microsoft/vscode-vsce#908 and microsoft/vscode#195559, it
appears that extensions should no longer be published for the now
removed win32-ia32 target.
@github-actions github-actions bot locked and limited conversation to collaborators Nov 30, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
engineering VS Code - Build / issue tracking / etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants