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

[PLAT-6339] Add binaryArch and runningOnRosetta to app metadata #1073

Merged
merged 2 commits into from
Apr 16, 2021

Conversation

nickdowell
Copy link
Contributor

@nickdowell nickdowell commented Apr 15, 2021

Goal

Allows Mac developers to identify which code slice from a Universal binary was running when an error occurred, and identify errors that may be specific to Rosetta.

Changeset

Adds binaryArch and runningOnRosetta to the app metadata.

Removes BSG_KSSystemField_BinaryCPUType and BSG_KSSystemField_BinaryCPUSubType since they are unused.

Testing

Manually tested using example apps and verified new fields appear on dashboard.

Amended E2E test scenario to verify presence of binaryArch

@github-actions
Copy link

github-actions bot commented Apr 15, 2021

Infer: No issues found 🎉

OCLint: No issues found 🎉

Bugsnag.framework binary size did not change - 1,119,520 bytes

Generated by 🚫 Danger

@nickdowell nickdowell force-pushed the nickdowell/binary-arch branch from 8db6d1e to 12d6302 Compare April 15, 2021 10:28
@nickdowell nickdowell marked this pull request as ready for review April 15, 2021 12:10
@nickdowell nickdowell merged commit 1a6a172 into next Apr 16, 2021
@nickdowell nickdowell deleted the nickdowell/binary-arch branch April 16, 2021 12:12
@nickdowell nickdowell mentioned this pull request Apr 21, 2021
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.

2 participants