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.

Sign up for GitHub

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

Jump to bottom

[Swift APIView] Migrate SwiftAPIView to use swift-syntax instead of swift-ast #3339

Closed
tjprescott opened this issue May 12, 2022 · 2 comments · Fixed by #4789
Closed

[Swift APIView] Migrate SwiftAPIView to use swift-syntax instead of swift-ast #3339

tjprescott opened this issue May 12, 2022 · 2 comments · Fixed by #4789
Assignees
@tjprescott
Labels
APIView Priority 1 APIView Swift

Comments

@tjprescott
Copy link
Member

Swift-syntax (https://github.com/apple/swift-syntax) is the official Apple-supported library for parsing Swift. We have had numerous issues arise due to relying on swift-ast, which has not been maintained since 2019. We should investigate refactoring SwiftAPIView to use swift-syntax.
@tjprescott
Copy link
Member Author

We should avoid any heavy refactoring until #2325 is complete.

@tjprescott tjprescott self-assigned this Nov 23, 2022
@tjprescott
Copy link
Member Author

https://dev.srdanstanic.com/swiftsyntax-tips/

@tjprescott tjprescott mentioned this issue Nov 23, 2022
Merged
2 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@tjprescott tjprescott

Labels
APIView Priority 1 APIView Swift
Projects
ApiView
Status: ✅ Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[Swift APIView] Migrate from SwiftAST to SwiftSyntax Azure/azure-sdk-tools
1 participant
@tjprescott