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

Add support for Mojo #831

Merged
merged 1 commit into from
May 26, 2024
Merged

Add support for Mojo #831

merged 1 commit into from
May 26, 2024

Conversation

jackos
Copy link
Contributor

@jackos jackos commented May 25, 2024

I've added support for the Mojo programming language: https://github.com/modularml/mojo

There was an existing interface definition language named Mojo, but the language itself is Mojom, so updated that as well: https://chromium.googlesource.com/chromium/src/+/HEAD/mojo/public/tools/bindings/README.md

I've added support for the Mojo programming language:
https://github.com/modularml/mojo

There was an existing interface definition language named Mojo, but the
language itself is Mojom, so updated that as well:
https://chromium.googlesource.com/chromium/src/+/HEAD/mojo/public/tools/bindings/README.md
@jackos jackos force-pushed the jackos/add-mojo branch from 642375e to ae61d65 Compare May 25, 2024 17:08
@AlDanial AlDanial merged commit 61d4fc9 into AlDanial:master May 26, 2024
1 check failed
@AlDanial
Copy link
Owner

Google Mojo docs are confusing. I couldn't tell if Mojo is a separate language from Mojom. Whatever, your renaming works for me.

The tests in this PR don't pass but I'll fix those in a separate commit.

Thanks.

AlDanial added a commit that referenced this pull request May 26, 2024
@jackos
Copy link
Contributor Author

jackos commented May 26, 2024

Epic thanks @AlDanial, I was having trouble installing the required perl modules on macos for the tests. Was going to fix after checking in CI, but very much appreciate you doing this for me!

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