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 archType and isARM() from site.js (Fixes #14404) #14409

Merged
merged 1 commit into from
Jul 10, 2024

Conversation

alexgibson
Copy link
Member

@alexgibson alexgibson commented Apr 4, 2024

One-line summary

We no longer actively use this code since #14012 was completed, so we can remove these functions from site.js.

Issue / Bugzilla link

#14404

Testing

  • Verify that nothing seems to be using either window.site.archType or window.site.isARM().
@alexgibson alexgibson added P3 Third level priority - Nice to have Needs Review Awaiting code review Review: XS Code review time: up to 30min labels Apr 4, 2024
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 4 times, most recently from 39ba637 to 9935deb Compare April 10, 2024 08:19
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 3 times, most recently from ea57bd7 to 23ca20a Compare April 23, 2024 14:47
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 2 times, most recently from 55fe891 to 61bdca5 Compare May 1, 2024 08:44
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 2 times, most recently from 702fbe0 to 0f67b67 Compare May 24, 2024 09:00
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 2 times, most recently from beb4082 to c164102 Compare June 3, 2024 13:18
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 5 times, most recently from 5add1bc to 391ba53 Compare June 17, 2024 08:52
@alexgibson alexgibson force-pushed the remove-arch-type-detection branch 2 times, most recently from 7fe6a3c to defbf7d Compare June 19, 2024 09:24
@alexgibson alexgibson added the Frontend HTML, CSS, JS... client side stuff label Jun 24, 2024
@wen-2018 wen-2018 self-assigned this Jul 10, 2024
@wen-2018 wen-2018 removed the Needs Review Awaiting code review label Jul 10, 2024
@wen-2018 wen-2018 merged commit d4a7f61 into mozilla:main Jul 10, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Frontend HTML, CSS, JS... client side stuff P3 Third level priority - Nice to have Review: XS Code review time: up to 30min
3 participants