Add ProtocolReader #7
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
omochi
approved these changes
Nov 5, 2021
| @@ -5,6 +5,20 @@ import PackageDescription | |||
| var dependencies: [Package.Dependency] = [] | |||
| var targetDependencies: [Target.Dependency] = [] | |||
|
|
|||
| #if swift(>=5.5) | |||
There was a problem hiding this comment.
Swift5.5/Xcode13.0環境では BinarySwiftSyntax を使わずに SwiftSyntax を使う状態になった？
これだと、未来のXcode14や、場合によってはXcode13.1でも動かなくなるので、
(指定してるSwiftSyntaxライブラリのバージョンと、ビルド環境が揃っていなければならない)
BinarySwiftSyntax も更新して、
SwiftSytanx-Xcode13.0 ターゲットを提供するようにしたいですね。
このパッチのマージ後にいつかやるでいいですが。
There was a problem hiding this comment.
そのとおりだと思います。BinarySwiftSyntaxの作り方がよくわかっていなくてここは一旦サボりました
| import Foundation | ||
|
|
||
| public struct FunctionRequirement { | ||
| public struct InputParameter { |
There was a problem hiding this comment.
FunctionにおいてParameterというと仮引数の事なので、「Input」というprefixは冗長なので消したいです。
Co-authored-by: omochimetaru <omochi.metaru@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
概要
protocolの読み取りに対応します。これにより、
protocolをベースに実質的な関数型やクラス型を読み取ってコード生成し、TypeScriptに関数型を提供することが狙いです。型としての関数型やタプルには既存実装と同様に対応していません。
associatedtypeに対しても型名を読み取るだけです。実用上必要なタイミングがあれば型制約の部分の読み取りも実装します。レビューしてほしいところ
FunctionRequirementPropertyRequirementまわりの命名や保持するプロパティに違和感がないかその他
また
asyncなどの新しいキーワードを扱う都合上、Xcode12ではビルドできなくなります。（おそらくSwiftSyntaxの
asyncOrReasyncKeywordが存在しないのでエラーになる）実用上問題ないかと思っているためXcode12への後方互換は気にしていない実装になっています。