Skip to content

Conversation

@jmarrec
Copy link
Collaborator

@jmarrec jmarrec commented Sep 18, 2023

@jmarrec
Copy link
Collaborator Author

jmarrec commented Sep 18, 2023

@wenyikuang, If you want to merge this one, the mac M1 one is hosted on my google drive. I'll put the link on slack.

@wenyikuang
Copy link
Contributor

wenyikuang commented Sep 19, 2023

Got the Mac arm64 build from Julien. Will update this PR and the one in SDK together once I find the place to upload.

@jmarrec
Copy link
Collaborator Author

jmarrec commented Sep 19, 2023

@wenyikuang S3:openstudio-resources/dependencies/

The PR builds go to s3:openstudio-sdk-dependencies/<runner-name>/<PR-number>

@jmarrec
Copy link
Collaborator Author

jmarrec commented Sep 20, 2023

@wenyikuang I don't have write access to this repo and need one approving reviewer. Seems you also don't. Pinging @tijcolem to approve.

Copy link
Contributor

@wenyikuang wenyikuang left a comment

Choose a reason for hiding this comment

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

Since we are conusuming it in OS here. We could just merge it.

@wenyikuang wenyikuang merged commit 8dd2055 into develop Oct 11, 2023
@wenyikuang wenyikuang deleted the 4969_json_schemer branch October 11, 2023 22:29
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.

4 participants