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

Render Swift declarations for Objective-C declarations #530

Merged
merged 4 commits into from
Apr 20, 2016

Conversation

jpsim
Copy link
Collaborator

@jpsim jpsim commented Apr 18, 2016

This points SourceKitten to jpsim/SourceKitten#95 in order to render the equivalent Swift declarations when generating Objective-C docs. I might not be able to revisit this for a little while, so if anyone else wants to pitch in, you're certainly welcome to do so /cc @esad @jverkoey @orta @jessesquires

Integration specs diff is here: https://github.com/realm/jazzy-integration-specs/compare/jp-render-swift-declarations-in-objc

For some reason (probably unrelated updates to SourceKitten), some extensions docs are now missing, which needs some investigation, hence the call for help above.

@jpsim jpsim changed the title render Swift declarations for Objective-C declarations [WIP] render Swift declarations for Objective-C declarations Apr 18, 2016
name: item.name,
abstract: render_markdown(abstract),
declaration: item.declaration,
other_language_declaration: item.other_language_declaration,
Copy link
Collaborator

Choose a reason for hiding this comment

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

to be generic, this should probably be a hash, languages to their ddeclarations

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

happy to do that when we get there. for now, this is a bit simpler.

@jpsim jpsim changed the title [WIP] render Swift declarations for Objective-C declarations Render Swift declarations for Objective-C declarations Apr 20, 2016
@jpsim
Copy link
Collaborator Author

jpsim commented Apr 20, 2016

Here's what Objective-C docs looks like with this PR:

image

@jpsim jpsim merged commit db0c549 into master Apr 20, 2016
@jpsim jpsim deleted the jp-render-swift-declarations-in-objc branch April 20, 2016 15:26
1ec5 added a commit to mapbox/mapbox-gl-native that referenced this pull request Jun 14, 2016
@pigeondev2 pigeondev2 modified the milestone: The Past Nov 22, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants