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

fix: add new macos examples #2372

Merged
merged 31 commits into from
Jul 29, 2024
Merged

fix: add new macos examples #2372

merged 31 commits into from
Jul 29, 2024

Conversation

bohdanprog
Copy link
Member

Summary

We want to avoid problems with GitHub workflows.
Create a new Macos example for new and old architecture.

Test Plan

Run GitHub workflows

Compatibility

OS Implemented
macOS

@bohdanprog bohdanprog changed the title Fix/add new macos examples fix: add new macos examples Jul 29, 2024
@@ -156,7 +158,7 @@ function parseTransformProp(
transformArray.push(`matrix(${transform.join(' ')})`);
} else {
const stringifiedProps = transformsArrayToProps(
transform as TransformsStyle['transform']
transform as TransformsStyleArray
Copy link
Member Author

Choose a reason for hiding this comment

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

The Test JS build / build (pull_request) build crashed without altering that type.

@bohdanprog bohdanprog marked this pull request as ready for review July 29, 2024 07:16
@bohdanprog bohdanprog self-assigned this Jul 29, 2024
Copy link
Member

@WoLewicki WoLewicki left a comment

Choose a reason for hiding this comment

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

If it works, then I'm fine with it for now :D

src/ReactNativeSVG.web.ts Outdated Show resolved Hide resolved
@bohdanprog bohdanprog merged commit 4fd2ccd into main Jul 29, 2024
3 checks passed
@bohdanprog bohdanprog deleted the fix/add-new-macos-examples branch July 29, 2024 12:46
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.

3 participants