Skip to content

Update on SwiftJavaDocumentation #372

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

Open
wants to merge 7 commits into
base: main
Choose a base branch
from

Conversation

koloyyee
Copy link
Contributor

Description

  • Change to "to" in SupportedFeatures.md JExtract Java -> Swift section
  • Added "watch" to "you may want to this"
  • Adding an introduction based on the WWDC video

@@ -38,7 +38,7 @@ Java `native` functions. JavaKit simplifies the type conversions

### JExtract: Java -> Swift

SwiftJava's `swift-java jextract` tool automates generating Java bindings from Swift sources.
SwiftJava's `swift-java jextract` tool automates generating Java bindings to Swift sources.
Copy link
Collaborator

Choose a reason for hiding this comment

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

This is "from"; we take swift sources as input and generate java; so it's not that the bindings are 'to' swift but that we generate them "from swift sources" 🤔

Maybe this whole sentence needs rewording?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah, ok, I was trying to align with the README line 34

**swift-java jextract** 

Is a source generator which will **generate Java bindings to existing Swift libraries**. 

How about changing the README to "from" for now, and rewording them both afterwards

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